tweak eslint rules

#181

.eslintignore

@@ -1,5 +1,6 @@
-__tests__/
+#__tests__/
 coverage/
 interop/
-*.test.js
-*.test.mjs
+examples/
+#*.test.js
+#*.test.mjs

.eslintrc.yml

@@ -2,6 +2,7 @@
 env:
   es6: true
   node: true
+  jest: true
   browser: false
 extends: airbnb-base
 parserOptions:
@@ -19,14 +20,22 @@ rules:
   semi:
   - error
   - never
+  comma-dangle:
+  - error
+  - never
   prefer-template: error
+  prefer-promise-reject-errors: off
   array-bracket-spacing:
   - error
   - always
   camelcase: warn
+  no-confusing-arrow: off
   no-useless-escape: warn
-  no-nested-ternary: warn
-  no-sequences: warn
-  no-plusplus: warn
+  no-nested-ternary: off
+  no-sequences: off
+  no-plusplus: off
+  no-shadow: off
+  implicit-arrow-linebreak: off
   import/no-extraneous-dependencies: warn
+  import/prefer-default-export: off
   max-len: warn

_internal/__tests__/call.test.js

@@ -5,13 +5,13 @@ describe('function/call', () => {
 
   test('sync', () => {
     const expected = 888
-    const actual = apply (increase) (887)
+    const actual = apply(increase)(887)
     expect(actual).toBe(expected)
   })
 
   test('async', () => {
     const expected = 888
-    const actual = apply (increase) (Promise.resolve(887))
+    const actual = apply(increase)(Promise.resolve(887))
     expect(actual).resolves.toBe(expected)
   })
-})
+})

_internal/__tests__/iterableSerialReduce.test.js

@@ -11,31 +11,31 @@ describe('internal/iterableSerialReduce', () => {
 
   test('sync array', () => {
     const expected = 6
-    const actual = iterableSerialReduce(add, 0, [1, 2, 3])
+    const actual = iterableSerialReduce(add, 0, [ 1, 2, 3 ])
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('sync array 2', () => {
     const expected = 6
-    const actual = iterableSerialReduce(add, null, [2, 3], Promise.resolve(1))
+    const actual = iterableSerialReduce(add, null, [ 2, 3 ], Promise.resolve(1))
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('sync iterator', () => {
     const expected = 6
     const actual = iterableSerialReduce(add, 0, iterator())
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('async array', () => {
     const expected = 6
-    const actual = iterableSerialReduce(asyncAdd, 0, [1, 2, 3])
+    const actual = iterableSerialReduce(asyncAdd, 0, [ 1, 2, 3 ])
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('async iterator', () => {
     const expected = 6
     const actual = iterableSerialReduce(asyncAdd, 0, iterator())
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 })

_internal/__tests__/iterableSerialReduceWhile.test.js

@@ -3,7 +3,7 @@ const iterableSerialReduceWhile = require('../iterableSerialReduceWhile')
 describe('internal/iterableSerialReduceWhile', () => {
   const add = (x, y) => x + y
   const asyncAdd = (x, y) => Promise.resolve().then(() => x + y)
-  const predicate = acc => x => acc <= 2
+  const predicate = acc => () => acc <= 2
   function* iterator() {
     yield 1
     yield 2
@@ -12,31 +12,31 @@ describe('internal/iterableSerialReduceWhile', () => {
 
   test('sync array', () => {
     const expected = 3
-    const actual = iterableSerialReduceWhile(predicate, add, 0, [1, 2, 3])
+    const actual = iterableSerialReduceWhile(predicate, add, 0, [ 1, 2, 3 ])
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('sync array 2', () => {
     const expected = 3
-    const actual = iterableSerialReduceWhile(predicate, add, null, [2, 3], Promise.resolve(1))
+    const actual = iterableSerialReduceWhile(predicate, add, null, [ 2, 3 ], Promise.resolve(1))
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('sync iterator', () => {
     const expected = 3
     const actual = iterableSerialReduceWhile(predicate, add, 0, iterator())
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('async array', () => {
     const expected = 3
-    const actual = iterableSerialReduceWhile(predicate, asyncAdd, 0, [1, 2, 3])
+    const actual = iterableSerialReduceWhile(predicate, asyncAdd, 0, [ 1, 2, 3 ])
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 
   test('async iterator', () => {
     const expected = 3
     const actual = iterableSerialReduceWhile(predicate, asyncAdd, 0, iterator())
     expect(actual).resolves.toBe(expected)
-  })  
+  })
 })

_internal/call.js

@@ -1,12 +1,12 @@
-
 const isThenable = require('../_internal/isThenable')
 
 /*
  * Takes a value or a Promise and applies func to it.
  */
 // call :: Function -> Any -> Any | Promise<Any>
-const call = func => value => (isThenable(value)
-  ? value.then(func)
-  : func(value))
+const call = func => value =>
+  isThenable(value)
+    ? value.then(func)
+    : func(value)
 
 module.exports = call

_internal/ensureExecutable.js

@@ -1,4 +1,3 @@
-
 const K = require('../combinators/K')
 const is = require('../type/is')
 

_internal/escapeRegExp.js

@@ -1,5 +1,5 @@
 /* from: https://stackoverflow.com/a/1144788/504836 */
 
-const escapeRegExp = str => str.replace(/([.*+?^=!:${}()|\[\]\/\\])/g, '\\$1')
+const escapeRegExp = str => str.replace(/([.*+?^=!:${}()|[\]/\\])/g, '\\$1')
 
 module.exports = escapeRegExp

_internal/iterableSerialReduce.js

@@ -1,4 +1,3 @@
-
 const iterableSerialReduceWhile = require('./iterableSerialReduceWhile')
 
 const iterableSerialReduce = iterableSerialReduceWhile.bind(null, null)

_internal/iterableSerialReduceWhile.js

@@ -1,4 +1,3 @@
-
 const iterableSerialReduceWhile = async (
   predicate,
   func,

_internal/maybeExec.js

@@ -1,4 +1,3 @@
-
 const is = require('../type/is')
 
 const isFunction = is(Function)

bin/patch-readme/index.mjs

@@ -6,7 +6,7 @@ import main from './main'
 const dependencies = {
   baseUrl: 'https://github.com/joelnet/MojiScript/tree/master',
   readFile: readFileUtf8,
-  log,
+  log
 }
 const state = `${process.cwd()}/README.md`
 

bin/patch-readme/main.mjs

@@ -4,13 +4,13 @@ import W from '../../combinators/W'
 
 const prependLinksInReadme = baseUrl => W(readme => pipe([
   getInternalLinks,
-  prependAllLinks(baseUrl)(readme),
+  prependAllLinks(baseUrl)(readme)
 ]))
 
 const main = ({ log, readFile, baseUrl }) => pipe([
   readFile,
   prependLinksInReadme(baseUrl),
-  log,
+  log
 ])
 
 export default main

bin/patch-readme/markdown.mjs

@@ -1,4 +1,4 @@
-import S from 'sanctuary'
+import S from 'sanctuary' // eslint-disable-line import/no-extraneous-dependencies
 import pipe from '../../core/pipe'
 import replace from '../../string/replace'
 
@@ -8,12 +8,12 @@ const isInternalLink = ([ , link ]) => !S.test(rxExternalLink)(link)
 
 export const getAllLinks = pipe([
   S.matchAll(rxLink),
-  S.map(({ groups }) => S.justs(groups)),
+  S.map(({ groups }) => S.justs(groups))
 ])
 
 export const getInternalLinks = pipe([
   getAllLinks,
-  S.filter(isInternalLink),
+  S.filter(isInternalLink)
 ])
 
 export const prependLink = baseUrl => document => ([ text, link ]) => replace(`[${text}](${link})`)(`[${text}](${baseUrl}/${link})`)(document)

combinators/__tests__/I.test.js

@@ -3,7 +3,7 @@ const I = require('../I')
 describe('combinators/I', () => {
   test('returns value', () => {
     const expected = 888
-    const actual = I (expected)
+    const actual = I(expected)
     expect(actual).toBe(expected)
   })
 })

combinators/__tests__/K.test.js

@@ -3,7 +3,7 @@ const K = require('../K')
 describe('combinators/K', () => {
   test('returns value', () => {
     const expected = 888
-    const actual = K (expected) (666)
+    const actual = K(expected)(666)
     expect(actual).toBe(expected)
   })
 })

combinators/__tests__/S.test.js

@@ -5,7 +5,7 @@ describe('combinators/S', () => {
     const expected = 30
     const f = a => b => a + b
     const g = a => a * 2
-    const actual = S (f) (g) (10)
+    const actual = S(f)(g)(10)
     expect(actual).toBe(expected)
   })
 })

combinators/__tests__/W.test.js

@@ -4,7 +4,7 @@ describe('combinators/W', () => {
   test('returns value', () => {
     const expected = 888
     const f = a => b => a + b
-    const actual = W (f) (444)
+    const actual = W(f)(444)
     expect(actual).toBe(expected)
   })
 })

console/__tests__/error.test.js

@@ -6,14 +6,14 @@ describe('console/error', () => {
 
   test('calls console.error', () => {
     const expected = 888
-    error (expected)
+    error(expected)
     const actual = global.console.error.mock.calls[0][0]
     expect(actual).toBe(expected)
   })
 
   test('returns original value', () => {
     const expected = 888
-    const actual = error (expected)
+    const actual = error(expected)
     expect(actual).toBe(expected)
   })
 })

console/__tests__/log.test.js

@@ -6,14 +6,14 @@ describe('console/log', () => {
 
   test('calls console.log', () => {
     const expected = 888
-    log (expected)
+    log(expected)
     const actual = global.console.log.mock.calls[0][0]
     expect(actual).toBe(expected)
   })
 
   test('returns original value', () => {
     const expected = 888
-    const actual = log (expected)
+    const actual = log(expected)
     expect(actual).toBe(expected)
   })
 })

console/__tests__/logF.test..js

@@ -8,14 +8,14 @@ describe('console/logF', () => {
 
   test('calls console.log', () => {
     const expected = 888
-    logF (double) (444)
+    logF(double)(444)
     const actual = global.console.log.mock.calls[0][0]
     expect(actual).toBe(expected)
   })
 
   test('returns original value', () => {
     const expected = 888
-    const actual = logF (double) (expected)
+    const actual = logF(double)(expected)
     expect(actual).toBe(expected)
   })
 })

console/error.js

@@ -1,6 +1,5 @@
-
 const tap = require('../function/tap')
 
-const error = tap(x => console.error(x))
+const error = tap(x => console.error(x)) // eslint-disable-line no-console
 
 module.exports = error

console/log.js

@@ -1,6 +1,5 @@
-
 const tap = require('../function/tap')
 
-const log = tap(x => console.log(x))
+const log = tap(x => console.log(x)) // eslint-disable-line no-console
 
 module.exports = log

console/logF.js

@@ -1,6 +1,5 @@
-
 const tap = require('../function/tap')
 
-const logF = func => tap(x => console.log(func(x)))
+const logF = func => tap(x => console.log(func(x))) // eslint-disable-line no-console
 
 module.exports = logF

core/__tests__/after.test.js

@@ -6,7 +6,7 @@ describe('core/after', () => {
     const expected = 888
     const f = jest.fn()
     const g = jest.fn()
-    await after (f) (g) (expected)
+    await after(f)(g)(expected)
     const actual = f.mock.calls[0][0]
     expect(actual).toBe(expected)
   })
@@ -16,7 +16,7 @@ describe('core/after', () => {
     const expected = 888
     const f = jest.fn()
     const g = jest.fn()
-    await after (f) (g) (expected)
+    await after(f)(g)(expected)
     const actual = g.mock.calls[0][0]
     expect(actual).toBe(expected)
   })
@@ -26,7 +26,7 @@ describe('core/after', () => {
     const expected = 888
     const f = jest.fn()
     const g = x => x * 2
-    const actual = after (f) (g) (444)
+    const actual = after(f)(g)(444)
     expect(actual).resolves.toBe(expected)
   })
 })