Removed validation build step

microsoft · Nov 5, 2018 · ca0a5a2 · ca0a5a2
1 parent d693a7a
commit ca0a5a2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 26 deletions.
diff --git a/build-tasks/common/meta.js b/build-tasks/common/meta.js
@@ -80,23 +80,7 @@ function getContribRuleNames() {
         .sort();
 }
 
-function getAllFormatterNames() {
-    const convertToFormatterNames = filename => {
-        filename = filename
-            .replace(/Formatter\..*/, '') // file extension plus Formatter suffix
-            .replace(/.*\//, ''); // leading path
-
-        return kebabCase(filename);
-    };
-
-    const formatters = glob.sync('src/*Formatter.ts').map(convertToFormatterNames);
-    formatters.sort();
-
-    return formatters;
-}
-
 module.exports = {
-    getAllFormatterNames,
     getContribRuleNames,
     getAllRules,
     getMetadataFromFile,

diff --git a/build-tasks/validate-documentation.js b/build-tasks/validate-documentation.js
@@ -4,11 +4,10 @@
  */
 
 const { yellowBright } = require('chalk');
-const { readFile, readJSON } = require('./common/files');
-const { getAllFormatterNames, getContribRuleNames } = require('./common/meta');
+const { readFile } = require('./common/files');
+const { getContribRuleNames } = require('./common/meta');
 
 const readmeText = readFile('README.md');
-const packageJson = readJSON('package.json');
 const validationErrors = [];
 
 getContribRuleNames().forEach(ruleName => {
@@ -17,13 +16,6 @@ getContribRuleNames().forEach(ruleName => {
     }
 });
 
-getAllFormatterNames().forEach(formatterName => {
-    if (readmeText.indexOf(formatterName) === -1) {
-        validationErrors.push('A formatter was found that is not documented in README.md: ' + formatterName);
-        validationFailed = true;
-    }
-});
-
 if (validationErrors.length > 0) {
     console.log(yellowBright(validationErrors.join('\n')));
     process.exit(1);