Responding to comments

Author: jessebeach

scripts/addRuleToIndex.js

@@ -9,8 +9,8 @@ export default function transformer(file, api, options) {
   } = options || {};
 
   const nameSort = (a, b) => {
-    const aName = a.key.value;
-    const bName = b.key.value;
+    const aName = (a.key.type === 'Literal') ? a.key.value : a.key.name;
+    const bName = (b.key.type === 'Literal') ? b.key.value : b.key.name;
     if (aName < bName) {
       return -1;
     }

scripts/create-rule.js

@@ -33,10 +33,12 @@ fs.writeFileSync(docsPath, docBoilerplate);
 
 // Add the rule to the index
 exec([
-  './node_modules/jscodeshift/bin/jscodeshift.sh',
+  path.join(
+    require.resolve('jscodeshift'),
+    require('jscodeshift/package.json').bin.jscodeshift
+  ),
   './src/index.js',
   '-t ./scripts/addRuleToIndex.js',
-  '-p',
   '--extensions js',
   '--parser flow',
   `--ruleName=${ruleName}`,
@@ -47,7 +49,5 @@ exec([
       console.error(`exec error: ${error}`);
       return;
     }
-    console.log(`stdout: ${stdout}`);
-    console.log(`stderr: ${stderr}`);
   }
 );

src/index.js

@@ -17,7 +17,7 @@ module.exports = {
     'img-has-alt': require('./rules/img-has-alt'),
     'img-redundant-alt': require('./rules/img-redundant-alt'),
     'label-has-for': require('./rules/label-has-for'),
-    'lang': require('./rules/lang'),
+    lang: require('./rules/lang'),
     'mouse-events-have-key-events': require('./rules/mouse-events-have-key-events'),
     'no-access-key': require('./rules/no-access-key'),
     'no-autofocus': require('./rules/no-autofocus'),
@@ -28,7 +28,7 @@ module.exports = {
     'onclick-has-role': require('./rules/onclick-has-role'),
     'role-has-required-aria-props': require('./rules/role-has-required-aria-props'),
     'role-supports-aria-props': require('./rules/role-supports-aria-props'),
-    'scope': require('./rules/scope'),
+    scope: require('./rules/scope'),
     'tabindex-no-positive': require('./rules/tabindex-no-positive'),
   },
   configs: {