Various clean up tasks (#473)

* Dropped Rachel Wang from docs reviewers

* Removing dependencies from no test message

I dropped the formatting dependencies from notest.js in the icons
package as they were causing an error about dynamic importing.

* Removing inconsistent use of AUTHORS files

* updating actions/checkout and actions/setup-node

* workflow: Changing Release to run on push, pull_request

Release should run on pushes to master and pull_requests from any
branch.

* workflows: Skip release if PR from fork

Adding a condition to skip the release job if the pull_request is from a
fork.

.github/CODEOWNERS

@@ -8,5 +8,4 @@
 
 
 *                           @cockroachdb/CRUX
-#TODO(nathanstilwell) - do we want to make this a team?
-/docs                       @annebirzin @rsadres @rachwangcrl
+/docs                       @annebirzin @rsadres

.github/workflows/pull-request-ci.yml

@@ -11,12 +11,12 @@ jobs:
         node-version: [16.x]
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
 
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v1
+      - uses: actions/setup-node@v3
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 16
+          cache: 'yarn'
 
       - name: Lerna bootstrap
         run: lerna bootstrap

.github/workflows/release.yml

@@ -1,12 +1,16 @@
 # Based on https://intuit.github.io/auto/docs/build-platforms/github-actions
 name: Release
 
-on: [push]
+on:
+  push:
+    branches:
+    - master
+  pull_request:
 
 jobs:
   release:
     runs-on: ubuntu-latest
-    if: "!contains(github.event.head_commit.message, 'ci skip') && !contains(github.event.head_commit.message, 'skip ci')"
+    if: "!contains(github.event.head_commit.message, 'ci skip') && !contains(github.event.head_commit.message, 'skip ci') && !github.event.pull_request.head.repo.fork"
     steps:
       - uses: actions/checkout@v3
         with:

packages/icons/notests.js

@@ -1,16 +1,5 @@
-const emoji = require("node-emoji");
-const chalk = require("chalk");
-
 console.log(`
-${emoji.emojify(
-  `:ghost: :ghost: :ghost: ${chalk.bold.red("Nope!")} :ghost: :ghost: :ghost:`,
-)}
-
 There are no tests here because this package contains no code.
-It is fully dependent on ${emoji.get("package")} ${chalk.bold(
-  "@svgr/cli",
-)} ${emoji.get("package")}, so you can look at their
-tests at ${chalk.cyanBright(
-  "https://github.com/gregberge/svgr/tree/master/packages/cli",
-)}.
+It is fully dependent on @svgr/cli. Please refer to their
+tests at https://github.com/gregberge/svgr/tree/master/packages/cli.
 `);

packages/icons/package.json

@@ -22,8 +22,7 @@
   "devDependencies": {
     "@svgr/cli": "5.3.0",
     "@types/react": "17.0.50",
-    "chalk": "5.1.2",
-    "node-emoji": "1.11.0",
+    "npm-run-all": "4.1.5",
     "react": "17.0.2",
     "rimraf": "3.0.2",
     "typescript": "4.8.4"

yarn.lock

@@ -6335,12 +6335,7 @@ [email protected]:
     ansi-styles "^4.1.0"
     supports-color "^7.1.0"
 
-[email protected]:
-  version "5.1.2"
-  resolved "https://registry.yarnpkg.com/chalk/-/chalk-5.1.2.tgz#d957f370038b75ac572471e83be4c5ca9f8e8c45"
-  integrity sha512-E5CkT4jWURs1Vy5qGJye+XwCkNj7Od3Af7CP6SujMetSMkLs8Do2RWJK5yx1wamHV/op8Rz+9rltjaTQWDnEFQ==
-
-chalk@^2.0.0, chalk@^2.3.2, chalk@^2.4.1, chalk@^2.4.2:
+chalk@^2.0.0, chalk@^2.4.1:
   version "2.4.2"
   resolved "https://registry.yarnpkg.com/chalk/-/chalk-2.4.2.tgz#cd42541677a54333cf541a49108c1432b44c9424"
   integrity sha512-Mti+f9lpJNcwF4tWV8/OrTTtF1gZi+f8FqlyAdouralcFWFQWF2+NgCHShjkCb+IFBLq9buZwE1xckQU4peSuQ==
@@ -11966,14 +11961,7 @@ node-dir@^0.1.10:
   dependencies:
     minimatch "^3.0.2"
 
-[email protected]:
-  version "1.11.0"
-  resolved "https://registry.yarnpkg.com/node-emoji/-/node-emoji-1.11.0.tgz#69a0150e6946e2f115e9d7ea4df7971e2628301c"
-  integrity sha512-wo2DpQkQp7Sjm2A0cq+sN7EHKO6Sl0ctXeBdFZrL9T9+UywORbufTcTZxom8YqpLQt/FqNMUkOpkZrJVYSKD3A==
-  dependencies:
-    lodash "^4.17.21"
-
-[email protected], node-fetch@^2.6.1, node-fetch@^2.6.7:
+node-fetch@^2.6.1, node-fetch@^2.6.7:
   version "2.6.7"
   resolved "https://registry.yarnpkg.com/node-fetch/-/node-fetch-2.6.7.tgz#24de9fba827e3b4ae44dc8b20256a379160052ad"
   integrity sha512-ZjMPFEfVx5j+y2yF35Kzx5sF7kDzxuDj6ziH4FFbOp87zKDZNx8yExJIb05OGF4Nlt9IHFIMBkRl41VdvcNdbQ==