chore: move 'prettier' settings to 'package.json' for format-on-save compatibility

'comma-dangle' has been modified in '.eslintrc.js' to get perfect compatibility
with 'prettier''s 'trailingComma: all' setting we use in 'package.json'.

Fixes #83.

Author: matteocng

.eslintrc.js

@@ -1,4 +1,5 @@
-module.exports = { // eslint-disable-line import/unambiguous
+// eslint-disable-next-line import/unambiguous
+module.exports = {
   parser: 'babel-eslint',
   plugins: [
     'ava', // SEE: https://github.com/avajs/eslint-plugin-ava
@@ -25,12 +26,11 @@ module.exports = { // eslint-disable-line import/unambiguous
     'plugin:import/warnings', // https://git.io/vyRv9
 
     // SEE: https://github.com/prettier/eslint-config-prettier
-    "prettier",
-    "prettier/flowtype",
-    "prettier/react",
+    'prettier',
+    'prettier/flowtype',
+    'prettier/react',
   ],
-  rules:
-  {
+  rules: {
     // Disable this for now, it doesn't seem to work correctly with polymorphic
     // Flow types (e.g <T>(...)).
     'arrow-parens': 0,
@@ -51,5 +51,16 @@ module.exports = { // eslint-disable-line import/unambiguous
 
     // No semicolons. Overrides the recommended 'airbnb' setting.
     semi: 0,
+
+    'comma-dangle': [
+      'error',
+      {
+        arrays: 'always-multiline',
+        objects: 'always-multiline',
+        imports: 'always-multiline',
+        exports: 'always-multiline',
+        functions: 'always-multiline',
+      },
+    ],
   },
 }

package.json

@@ -77,6 +77,14 @@
   "engines": {
     "node": ">=6"
   },
+  "prettier": {
+    "bracketSpacing": true,
+    "singleQuote": true,
+    "jsxBracketSameLine": true,
+    "trailingComma": "all",
+    "printWidth": 80,
+    "semi": false
+  },
   "scripts": {
     "build": "cross-env BABEL_ENV=build babel src --out-dir ./lib",
     "build:module": "cross-env BABEL_ENV=module babel src --out-dir ./module ",

scripts/tasks/prettier.js

@@ -16,31 +16,18 @@ const prettierCmd = path.resolve(
   __dirname,
   `../../node_modules/.bin/${prettier}`,
 )
-const defaultOptions = {
-  'bracket-spacing': 'true',
-  'single-quote': 'true',
-  'jsx-bracket-same-line': 'true',
-  'trailing-comma': 'all',
-  'print-width': 80,
-  semi: 'false',
-  parser: 'flow',
-}
 
 // We assign the 'Object' type to specify that 'config' is not sealed.
 // eslint-disable-next-line flowtype/no-weak-types
 let config: Object = {
   default: {
-    patterns: ['src/**/*.js'],
+    patterns: ['src/**/*.js', '.eslintrc.js'],
     ignore: ['**/node_modules/**'],
   },
   scripts: {
     patterns: ['scripts/**/*.js'],
     ignore: [],
   },
-  static: {
-    patterns: ['static/**/*.js'],
-    ignore: [],
-  },
 }
 
 if (process.argv[3]) {
@@ -67,9 +54,7 @@ Object.keys(config).forEach(key => {
     patterns.length > 1 ? `{${patterns.join(',')}}` : `${patterns.join(',')}`
   const files = glob.sync(globPattern, { ignore })
 
-  const args = Object.keys(defaultOptions).map(
-    k => `--${k}=${(options && options[k]) || defaultOptions[k]}`,
-  )
+  const args = []
   args.push(`--${shouldWrite ? 'write' : 'l'}`)
 
   try {