fix: upgrade to prettier@8 (#103)

Author: Kent C. Dodds

__mocks__/prettier-eslint.js

@@ -5,7 +5,7 @@
 // so we just return some spies here and assert
 // that we're calling prettier-eslint APIs correctly
 // eslint-disable-next-line require-await
-const format = jest.fn(async ({text, filePath = ''}) => {
+const format = jest.fn(({text, filePath = ''}) => {
   if (text === 'MOCK_SYNTAX_ERROR' || filePath.includes('syntax-error')) {
     throw new Error('Mock error for a syntax error')
   } else if (filePath.includes('eslint-config-error')) {

package.json

@@ -12,10 +12,11 @@
   "scripts": {
     "start": "nps",
     "test": "nps test",
-    "precommit":
-      "lint-staged && opt --in pre-commit --exec \"npm start validate\""
+    "precommit": "lint-staged && opt --in pre-commit --exec \"npm start validate\""
   },
-  "files": ["dist"],
+  "files": [
+    "dist"
+  ],
   "keywords": [],
   "author": "Kent C. Dodds <[email protected]> (http://kentcdodds.com/)",
   "license": "MIT",
@@ -35,7 +36,7 @@
     "lodash.memoize": "^4.1.2",
     "loglevel-colored-level-prefix": "^1.0.0",
     "messageformat": "^1.0.2",
-    "prettier-eslint": "^7.0.0",
+    "prettier-eslint": "^8.0.0",
     "rxjs": "^5.3.0",
     "yargs": "8.0.2"
   },
@@ -63,7 +64,10 @@
     "yargs-parser": "7.0.0"
   },
   "lint-staged": {
-    "*.+(js|json)": ["node ./dist/index --write", "git add"]
+    "*.+(js|json)": [
+      "node ./dist/index --write",
+      "git add"
+    ]
   },
   "repository": {
     "type": "git",

src/format-files.js

@@ -78,7 +78,7 @@ function formatFilesFromArgv({
 async function formatStdin(prettierESLintOptions) {
   const stdinValue = (await getStdin()).trim()
   try {
-    const formatted = await format({text: stdinValue, ...prettierESLintOptions})
+    const formatted = format({text: stdinValue, ...prettierESLintOptions})
     process.stdout.write(formatted)
     return Promise.resolve(formatted)
   } catch (error) {
@@ -197,18 +197,11 @@ function getFilesFromGlob(ignoreGlobs, applyEslintIgnore, fileGlob) {
 
 function formatFile(filePath, prettierESLintOptions, cliOptions) {
   const fileInfo = {filePath}
-  let format$ = rxReadFile(filePath, 'utf8').mergeMap(text => {
-    const promise = format({
-      text,
-      filePath,
-      ...prettierESLintOptions,
-    }).then(formatted => {
-      fileInfo.text = text
-      fileInfo.formatted = formatted
-      fileInfo.unchanged = fileInfo.text === fileInfo.formatted
-      return fileInfo
-    })
-    return Rx.Observable.fromPromise(promise)
+  let format$ = rxReadFile(filePath, 'utf8').map(text => {
+    fileInfo.text = text
+    fileInfo.formatted = format({text, filePath, ...prettierESLintOptions})
+    fileInfo.unchanged = fileInfo.text === fileInfo.formatted
+    return fileInfo
   })
 
   if (cliOptions.write) {
@@ -276,3 +269,4 @@ function getIsIgnored(filename) {
   instance.add(ignoreLines)
   return instance.ignores.bind(instance)
 }
+