Update dependency prettier to v2.3.0

Author: renovate-bot

.eslintrc

@@ -2,14 +2,8 @@
   "parserOptions": {
     "ecmaVersion": 2018
   },
-  "extends": [
-    "eslint:recommended",
-    "prettier"
-  ],
-  "plugins": [
-    "jest",
-    "prettier"
-  ],
+  "extends": ["eslint:recommended"],
+  "plugins": ["jest"],
   "env": {
     "node": true,
     "jest": true,
@@ -18,15 +12,5 @@
   "globals": {
     "any": false,
     "Promise": true
-  },
-  "rules": {
-    "prettier/prettier": ["error", {
-      "bracketSpacing": false,
-      "jsxBracketSameLine": true,
-      "parser": "flow",
-      "printWidth": 80,
-      "singleQuote": true,
-      "trailingComma": "all"
-    }]
   }
-}
+}

.github/workflows/ci.yml

@@ -15,4 +15,3 @@ jobs:
         with:
           node-version: '14'
       - run: npm run docker:test
-

.prettierignore

@@ -0,0 +1 @@
+_coverage

.prettierrc.json

@@ -0,0 +1,5 @@
+{
+  "bracketSpacing": false,
+  "singleQuote": true,
+  "trailingComma": "all"
+}

Dockerfile

@@ -1,6 +1,6 @@
-FROM node:16.1.0
+FROM node:14.16.1
 
-RUN npm install --global npm
+RUN npm install -g npm@latest
 
 RUN mkdir -p /usr/src/app
 WORKDIR /usr/src/app

README.md

@@ -7,11 +7,11 @@
 
 ## Table of Contents
 
-* [Background](#background)
-* [Getting Started](#getting-started)
-* [Running Tests](#running-tests)
-* [Documentation](#documentation)
-* [License](#license)
+- [Background](#background)
+- [Getting Started](#getting-started)
+- [Running Tests](#running-tests)
+- [Documentation](#documentation)
+- [License](#license)
 
 ## Background
 
@@ -57,4 +57,4 @@ $ java -jar target/realworld-server-tester-0.1.0-SNAPSHOT-standalone.jar http://
 
 ## License
 
-MIT
+MIT

bin/test.sh

@@ -3,6 +3,7 @@
 set -e
 
 if [ "$CI" = true ]; then
+  npm run check:format
   npm run lint
   npx jest -i --coverage
   bash <(curl -s https://codecov.io/bash)

codecov.yml

@@ -0,0 +1,4 @@
+comment: off
+coverage:
+  status:
+    patch: off

docker-compose.development.yml

@@ -14,4 +14,4 @@ services:
       - POSTGRES_DB=realworld_development
 
 volumes:
-  development_data:
+  development_data:

docker-compose.test.yml

@@ -21,4 +21,4 @@ services:
       - POSTGRES_DB=realworld_test
 
 volumes:
-  test_data:
+  test_data:

docker-compose.yml

@@ -20,4 +20,4 @@ services:
     image: postgres:13.2
     environment:
       - POSTGRES_PASSWORD=password
-      - POSTGRES_USER=realworld
+      - POSTGRES_USER=realworld

docs/structure/initializers.md

@@ -1,4 +1,4 @@
 # Initializers
 
 The initialization procedure is based on [the approach taken by the Locomotive framework](http://www.locomotivejs.org/guide/initialization/). App initialization steps have been split into separate functions in
-the `initializers/` folder, and are run in a certain order. Any required app-wide references are set on the [`app.locals`](https://expressjs.com/en/4x/api.html#app.locals) object.
+the `initializers/` folder, and are run in a certain order. Any required app-wide references are set on the [`app.locals`](https://expressjs.com/en/4x/api.html#app.locals) object.

docs/testing/outside-in.md

@@ -1,3 +1,3 @@
 # Outside-In
 
-The preference is to create high-level tests first in order to test the API end-to-end from a user’s perspective. From there we can drop down into lower-level tests like unit tests (this hasn't yet been done in this repo). See [Testing from the Outside-In](https://robots.thoughtbot.com/testing-from-the-outsidein) for more information.
+The preference is to create high-level tests first in order to test the API end-to-end from a user’s perspective. From there we can drop down into lower-level tests like unit tests (this hasn't yet been done in this repo). See [Testing from the Outside-In](https://robots.thoughtbot.com/testing-from-the-outsidein) for more information.

docs/testing/prefer-snapshot-tests.md

@@ -1,3 +1,3 @@
 # Prefer Snapshot Tests
 
-Where possible Jest's [snapshot testing](https://facebook.github.io/jest/docs/en/snapshot-testing.html) feature is used in order to validate key parts of the API response. Where this is not straightforward, for example when the response returns creation dates which vary over time, we've fallen back to more specific assertions.
+Where possible Jest's [snapshot testing](https://facebook.github.io/jest/docs/en/snapshot-testing.html) feature is used in order to validate key parts of the API response. Where this is not straightforward, for example when the response returns creation dates which vary over time, we've fallen back to more specific assertions.

docs/testing/run-in-band.md

@@ -1,3 +1,3 @@
 # Run in Band
 
-Since we're hitting a real DB via our functional tests, we want to run them serially in the current process using Jest's [`runInBand`](https://facebook.github.io/jest/docs/en/cli.html#runinband) option so they remain isolated. If the test suite gets expanded to include unit tests that don't hit the DB for example, then we may want to be specific about when we use that flag in order to keep the tests running as fast as possible.
+Since we're hitting a real DB via our functional tests, we want to run them serially in the current process using Jest's [`runInBand`](https://facebook.github.io/jest/docs/en/cli.html#runinband) option so they remain isolated. If the test suite gets expanded to include unit tests that don't hit the DB for example, then we may want to be specific about when we use that flag in order to keep the tests running as fast as possible.

lib/articles/__tests__/functional-test.js

@@ -5,14 +5,8 @@ const moment = require('moment');
 const request = require('supertest');
 const app = require('../../app');
 
-const {
-  articles,
-  articlesTags,
-  favorites,
-  followers,
-  tags,
-  users,
-} = app.locals.services;
+const {articles, articlesTags, favorites, followers, tags, users} =
+  app.locals.services;
 
 describe('articles', () => {
   let author;

lib/middleware/jwt-auth.js

@@ -4,39 +4,37 @@ const Boom = require('@hapi/boom');
 const errorCatcher = require('async-error-catcher').default;
 const passport = require('passport');
 
-const callback = (req, res, next) => async (
-  error,
-  {userId} = {},
-  info = {},
-) => {
-  const {
-    app: {
-      locals: {
-        services: {users},
+const callback =
+  (req, res, next) =>
+  async (error, {userId} = {}, info = {}) => {
+    const {
+      app: {
+        locals: {
+          services: {users},
+        },
       },
-    },
-    locals: {trx},
-  } = res;
+      locals: {trx},
+    } = res;
 
-  if (error) {
-    return next(error);
-  }
+    if (error) {
+      return next(error);
+    }
 
-  if (info instanceof Error || info.name === 'JsonWebTokenError') {
-    return next(Boom.unauthorized());
-  }
+    if (info instanceof Error || info.name === 'JsonWebTokenError') {
+      return next(Boom.unauthorized());
+    }
 
-  let user;
-  try {
-    user = await users.fetch({id: userId}, {transacting: trx});
-  } catch (error) {
-    return next(Boom.unauthorized());
-  }
+    let user;
+    try {
+      user = await users.fetch({id: userId}, {transacting: trx});
+    } catch (error) {
+      return next(Boom.unauthorized());
+    }
 
-  req.user = user;
+    req.user = user;
 
-  next();
-};
+    next();
+  };
 
 exports.required = errorCatcher(async (req, res, next) => {
   // prettier-ignore