Unify CI tests (#1004)

This does a few things to simplify testing and CI:

* `npm test` locally will now include format checking, which previously
  it did not. This means if you run `npm test` locally you can be fairly
  sure that CI will pass.
* Broke out CI jobs to run all test sub-steps in parallel. This should
  make it a little easier to understand why a PR failed CI.
* Remove the dedicated prettier workflow (in favor of the one in
  test/CI)
* Updates prettier and action versions

Author: leebyron

.github/workflows/ci.yml

@@ -1,32 +1,45 @@
 name: CI
+
 on:
   push:
     branches:
       - main
   pull_request:
-    branches:
-      - main
+
 jobs:
-  test:
+  test-spelling:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
-        with:
-          node-version: "16.x"
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+      - run: npm ci
+      - run: npm run test:spelling
+  test-format:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
       - run: npm ci
-      - run: npm test
+      - run: npm run test:format
+  test-build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+      - run: npm ci
+      - run: npm run test:build
   publish:
     if: github.ref == 'refs/heads/main'
-    needs: test
+    needs:
+      - test-spelling
+      - test-format
+      - test-build
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           fetch-depth: 0
-      - uses: actions/setup-node@v1
-        with:
-          node-version: "16.x"
+      - uses: actions/setup-node@v3
       - run: npm ci
       - run: npm run build
       - uses: peaceiris/actions-gh-pages@v3

.github/workflows/prettier.yaml

@@ -13,18 +13,18 @@
     "url": "http://github.com/graphql/graphql-spec.git"
   },
   "scripts": {
-    "test": "npm run test:build && npm run test:spellcheck",
-    "test:build": "spec-md --metadata spec/metadata.json spec/GraphQL.md > /dev/null",
-    "test:spellcheck": "cspell \"spec/**/*.md\" README.md",
+    "test": "npm run test:spelling && npm run test:format && npm run test:build",
+    "test:spelling": "cspell \"spec/**/*.md\" README.md",
     "format": "prettier --write \"**/*.{md,yml,yaml,json}\"",
-    "format:check": "prettier --check \"**/*.{md,yml,yaml,json}\"",
+    "test:format": "prettier --check \"**/*.{md,yml,yaml,json}\"",
     "build": "./build.sh",
+    "test:build": "spec-md --metadata spec/metadata.json spec/GraphQL.md > /dev/null",
     "watch": "nodemon -e json,md --exec \"npm run build\""
   },
   "devDependencies": {
     "cspell": "5.9.1",
     "nodemon": "2.0.12",
-    "prettier": "^2.3.0",
+    "prettier": "2.8.1",
     "spec-md": "3.0.2"
   },
   "prettier": {