chore: setup eslint and prettier

Author: satya164

.babelrc

@@ -6,5 +6,6 @@
       }
     }],
     "@babel/preset-typescript"
-  ]
+  ],
+  "plugins": ["@babel/plugin-proposal-optional-chaining"]
 }

.eslintignore

@@ -0,0 +1,3 @@
+node_modules/
+coverage/
+lib/

.eslintrc.json

@@ -0,0 +1,4 @@
+{
+  "extends": "satya164",
+  "env": { "node": true }
+}

bin/bob.js

@@ -1,3 +1,4 @@
 #!/usr/bin/env node
 
+// eslint-disable-next-line import/no-commonjs
 require('../lib/cli');

commitlint.config.js

@@ -1,5 +1,3 @@
-/* eslint-disable import/no-commonjs */
-
 module.exports = {
   extends: ['@commitlint/config-conventional'],
 };

package.json

@@ -18,6 +18,7 @@
     "registry": "https://registry.npmjs.org/"
   },
   "scripts": {
+    "lint": "eslint --ext '.js,.ts,.tsx' .",
     "typescript": "tsc --noEmit",
     "build": "babel --extensions .ts,.tsx src --out-dir lib --ignore '**/__tests__/**' --source-maps --delete-dir-on-start",
     "watch": "yarn build --watch",
@@ -41,6 +42,7 @@
   },
   "devDependencies": {
     "@babel/cli": "^7.7.7",
+    "@babel/plugin-proposal-optional-chaining": "^7.7.5",
     "@babel/preset-env": "^7.7.7",
     "@babel/preset-typescript": "^7.7.7",
     "@commitlint/config-conventional": "^8.2.0",
@@ -53,6 +55,8 @@
     "@types/inquirer": "^6.5.0",
     "@types/yargs": "^13.0.3",
     "commitlint": "^8.2.0",
+    "eslint": "^6.7.2",
+    "eslint-config-satya164": "^3.1.5",
     "husky": "^3.1.0",
     "prettier": "^1.19.1",
     "release-it": "^12.4.3",
@@ -61,7 +65,7 @@
   "husky": {
     "hooks": {
       "commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
-      "pre-commit": "yarn typescript"
+      "pre-commit": "yarn lint && yarn typescript"
     }
   }
 }

src/cli.ts

@@ -12,12 +12,15 @@ import buildModule from './targets/module';
 import buildTypescript from './targets/typescript';
 import { Options } from './types';
 
+// eslint-disable-next-line import/no-commonjs
 const { name } = require('../package.json');
+
 const root = process.cwd();
 const explorer = cosmiconfigSync(name);
 
 const FLOW_PRGAMA_REGEX = /\*?\s*@(flow)\b/m;
 
+// eslint-disable-next-line babel/no-unused-expressions
 yargs
   .command('init', 'configure the package to use bob', {}, async () => {
     const pak = path.join(root, 'package.json');
@@ -131,9 +134,7 @@ yargs
         const { replace } = await inquirer.prompt({
           type: 'confirm',
           name: 'replace',
-          message: `Your package.json has the '${key}' field set to '${
-            pkg[key]
-          }'. Do you want to replace it with '${entry}'?`,
+          message: `Your package.json has the '${key}' field set to '${pkg[key]}'. Do you want to replace it with '${entry}'?`,
           default: true,
         });
 
@@ -149,9 +150,7 @@ yargs
       const { replace } = await inquirer.prompt({
         type: 'confirm',
         name: 'replace',
-        message: `Your package.json has the 'scripts.prepare' field set to '${
-          pkg.scripts.prepare
-        }'. Do you want to replace it with '${prepare}'?`,
+        message: `Your package.json has the 'scripts.prepare' field set to '${pkg.scripts.prepare}'. Do you want to replace it with '${prepare}'?`,
         default: true,
       });
 
@@ -233,11 +232,9 @@ yargs
   .command('build', 'build files for publishing', {}, async argv => {
     const result = explorer.search();
 
-    if (!(result && result.config)) {
+    if (!result?.config) {
       logger.exit(
-        `No configuration found. Run '${
-          argv.$0
-        } init' to create one automatically.`
+        `No configuration found. Run '${argv.$0} init' to create one automatically.`
       );
     }
 

src/targets/aar.ts

@@ -7,15 +7,15 @@ import { Input } from '../types';
 import jetifier from '../utils/jetifier';
 
 type TargetOptions = {
-  androidPath: string,
+  androidPath: string;
   androidBundleName: string;
-  reverseJetify: boolean
+  reverseJetify: boolean;
 };
 
 const defaultOptions: TargetOptions = {
-  androidPath: "android",
-  androidBundleName: "android.aar",
-  reverseJetify: false
+  androidPath: 'android',
+  androidBundleName: 'android.aar',
+  reverseJetify: false,
 };
 
 type Options = Input & {
@@ -24,7 +24,10 @@ type Options = Input & {
 
 async function createGradleFile(file: string) {
   await fs.createFile(file);
-  await fs.writeFile(file, 'configurations.maybeCreate("default")\nartifacts.add("default", file(\'android.aar\'))')
+  await fs.writeFile(
+    file,
+    'configurations.maybeCreate("default")\nartifacts.add("default", file(\'android.aar\'))'
+  );
 }
 
 export default async function build({
@@ -35,7 +38,7 @@ export default async function build({
 }: Options) {
   const targetOptions = {
     ...defaultOptions,
-    ...options
+    ...options,
   };
 
   report.info(
@@ -44,18 +47,32 @@ export default async function build({
 
   await del([output]);
 
-  await androidAssemble({ root, androidPath: targetOptions.androidPath, report });
+  await androidAssemble({
+    root,
+    androidPath: targetOptions.androidPath,
+    report,
+  });
 
   report.info(
-    `Creating new output directory at ${chalk.blue(path.relative(root, output))}`
+    `Creating new output directory at ${chalk.blue(
+      path.relative(root, output)
+    )}`
   );
   await fs.mkdir(output);
 
-  const sourceAar = path.join(targetOptions.androidPath, 'build', 'outputs', 'aar', targetOptions.androidBundleName);
+  const sourceAar = path.join(
+    targetOptions.androidPath,
+    'build',
+    'outputs',
+    'aar',
+    targetOptions.androidBundleName
+  );
   const targetAar = path.join(output, targetOptions.androidBundleName);
 
   report.info(
-    `Copying AAR from ${chalk.blue(path.relative(root, sourceAar))} to ${chalk.blue(path.relative(root, targetAar))}`
+    `Copying AAR from ${chalk.blue(
+      path.relative(root, sourceAar)
+    )} to ${chalk.blue(path.relative(root, targetAar))}`
   );
   await fs.copyFile(sourceAar, targetAar);
 
@@ -68,31 +85,38 @@ export default async function build({
   if (targetOptions.reverseJetify) {
     const supportOutputPath = path.join(output, 'support');
     report.info(
-      `Creating new support output directory at ${chalk.blue(path.relative(root, supportOutputPath))}`
+      `Creating new support output directory at ${chalk.blue(
+        path.relative(root, supportOutputPath)
+      )}`
     );
     await fs.mkdir(supportOutputPath);
 
-    const supportAar = path.join(supportOutputPath, targetOptions.androidBundleName);
+    const supportAar = path.join(
+      supportOutputPath,
+      targetOptions.androidBundleName
+    );
     report.info(
-      `Using Jetifier to convert AAR from AndroidX to Support AAR at ${chalk.blue(path.relative(root, supportAar))}`
+      `Using Jetifier to convert AAR from AndroidX to Support AAR at ${chalk.blue(
+        path.relative(root, supportAar)
+      )}`
     );
 
     await jetifier({
       root,
       report,
       input: targetAar,
       output: supportAar,
-      reverse: true
+      reverse: true,
     });
 
     const supportGradleFile = path.join(supportOutputPath, 'build.gradle');
     report.info(
-      `Creating Support AAR Gradle file at ${chalk.blue(path.relative(root, supportGradleFile))}`
+      `Creating Support AAR Gradle file at ${chalk.blue(
+        path.relative(root, supportGradleFile)
+      )}`
     );
     await createGradleFile(supportGradleFile);
   }
 
-  report.success(
-    `Wrote files to ${chalk.blue(path.relative(root, output))}`
-  );
+  report.success(`Wrote files to ${chalk.blue(path.relative(root, output))}`);
 }

src/targets/commonjs.ts

@@ -28,7 +28,7 @@ export default async function build({
     options: {
       presets: [[require.resolve('metro-react-native-babel-preset')]],
     },
-    flow: options && options.flow ? true : false,
+    flow: options?.flow ? true : false,
     report,
   });
 }

src/targets/module.ts

@@ -33,7 +33,7 @@ export default async function build({
         ],
       ],
     },
-    flow: options && options.flow ? true : false,
+    flow: options?.flow ? true : false,
     report,
   });
 }