Fix: remove Node.js and CommonJS category from build process (#13242)

* Fix: Remove Node.js and CommonJS category from build process

* Remove categories that do not contain any rules

Author: Kai Cataldo

Makefile.js

@@ -210,6 +210,9 @@ function generateRuleIndexPage() {
             }
         });
 
+    // `.rules` will be `undefined` if all rules in category are deprecated.
+    categoriesData.categories = categoriesData.categories.filter(category => !!category.rules);
+
     const output = yaml.safeDump(categoriesData, { sortKeys: true });
 
     output.to(outputFile);

conf/category-list.json

@@ -4,7 +4,6 @@
         { "name": "Best Practices", "description": "These rules relate to better ways of doing things to help you avoid problems:" },
         { "name": "Strict Mode", "description": "These rules relate to strict mode directives:" },
         { "name": "Variables", "description": "These rules relate to variable declarations:" },
-        { "name": "Node.js and CommonJS", "description": "These rules relate to code running in Node.js, or in browsers with CommonJS:" },
         { "name": "Stylistic Issues", "description": "These rules relate to style guidelines, and are therefore quite subjective:" },
         { "name": "ECMAScript 6", "description": "These rules relate to ES6, also known as ES2015:" }
     ],