[lint] Add a "lint" script to all packages and fix lint warnings & er…

…rors This turns on linting for all of the packages, enforced by `expotools check-packages`. I added a "lint" script to each package and an `.eslintrc.js` file created by expo-module-scripts. For some packages like jest-expo that are intended to run on Node, I used a small handwritten ESLint config that uses the universe/node config. Tested with `expotools check-packages` after fixing all the lint warnings and errors.
yirenge · Aug 31, 2019 · 79a5fc6 · 79a5fc6
1 parent e4cc199
commit 79a5fc6
Show file tree

Hide file tree

Showing 327 changed files with 1,611 additions and 864 deletions.
diff --git a/packages/@unimodules/core/.eslintrc.js b/packages/@unimodules/core/.eslintrc.js
@@ -0,0 +1,2 @@
+// @generated by expo-module-scripts
+module.exports = require('expo-module-scripts/eslintrc.base.js');
diff --git a/packages/@unimodules/core/.npmignore b/packages/@unimodules/core/.npmignore
@@ -1,2 +1,2 @@
-// @generated by expo-module-scripts
+# @generated by expo-module-scripts
 babel.config.js
diff --git a/packages/@unimodules/core/build/deprecate.js b/packages/@unimodules/core/build/deprecate.js
diff --git a/packages/@unimodules/core/build/deprecate.js.map b/packages/@unimodules/core/build/deprecate.js.map
diff --git a/packages/@unimodules/core/package.json b/packages/@unimodules/core/package.json
@@ -8,6 +8,7 @@
   "scripts": {
     "build": "expo-module build",
     "clean": "expo-module clean",
+    "lint": "expo-module lint",
     "test": "expo-module test",
     "prepare": "expo-module prepare",
     "prepublishOnly": "expo-module prepublishOnly",

diff --git a/packages/@unimodules/core/src/deprecate.ts b/packages/@unimodules/core/src/deprecate.ts
@@ -58,6 +58,6 @@ function prependLibrary(library: string, message: string): string {
  * expo-ar -> EXPO_AR
  */
 function codeFromLibrary(library: string): string {
-  const code = library.replace(/[-\.]/g, '_').toUpperCase();
+  const code = library.replace(/[-.]/g, '_').toUpperCase();
   return code;
 }
diff --git a/packages/@unimodules/react-native-adapter/.eslintrc.js b/packages/@unimodules/react-native-adapter/.eslintrc.js
@@ -0,0 +1,2 @@
+// @generated by expo-module-scripts
+module.exports = require('expo-module-scripts/eslintrc.base.js');
diff --git a/packages/@unimodules/react-native-adapter/.npmignore b/packages/@unimodules/react-native-adapter/.npmignore
@@ -1,2 +1,2 @@
-// @generated by expo-module-scripts
+# @generated by expo-module-scripts
 babel.config.js
diff --git a/packages/@unimodules/react-native-adapter/build/NativeModulesProxy.js.map b/packages/@unimodules/react-native-adapter/build/NativeModulesProxy.js.map
diff --git a/packages/@unimodules/react-native-adapter/build/NativeModulesProxy.web.js.map b/packages/@unimodules/react-native-adapter/build/NativeModulesProxy.web.js.map
diff --git a/packages/@unimodules/react-native-adapter/build/index.js b/packages/@unimodules/react-native-adapter/build/index.js
diff --git a/packages/@unimodules/react-native-adapter/build/index.js.map b/packages/@unimodules/react-native-adapter/build/index.js.map
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		// @generated by expo-module-scripts
		module.exports = require('expo-module-scripts/eslintrc.base.js');