chore: remove mention of yarn

Switch root package manager to pnpm.
Remove clang tools, we'll just use prettier.
Remove stray reference to rules_codeowners which is already removed.

Author: alexeagle

.github/workflows/update-nodejs-versions.yml

@@ -5,7 +5,7 @@ on:
   workflow_dispatch:
   schedule:
     # run at 01:30 UTC daily
-    - cron: '30 1 * * *'
+    - cron: "30 1 * * *"
 
 permissions:
   contents: read
@@ -26,11 +26,11 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v2
         with:
-          node-version: '14.x'
-
-      - name: Run yarn update-nodejs-versions
+          node-version: "14.x"
+      - uses: pnpm/action-setup@v4
+      - name: Run update-nodejs-versions
         run: |
-          yarn run update-nodejs-versions
+          npm run update-nodejs-versions
           bazel run docs:update
 
       - name: Create Pull Request
@@ -40,7 +40,7 @@ jobs:
           commit-message: Update Node.js Versions
           title: Update Node.js Versions
           body: |
-            - Updated Node.js Versions using `yarn run update-nodejs-versions`
+            - Updated Node.js Versions using `npm run update-nodejs-versions`
 
             Auto-generated by [create-pull-request][1]
 

.pre-commit-config.yaml

@@ -9,7 +9,7 @@ default_stages: [commit]
 repos:
   # Check formatting and lint for starlark code
   - repo: https://github.com/keith/pre-commit-buildifier
-    rev: 6.4.0
+    rev: 7.3.1
     hooks:
       - id: buildifier
       - id: buildifier-lint

.prettierignore

@@ -1 +1,2 @@
 docs/*.md
+pnpm-lock.yaml

DEVELOPING.md

@@ -9,15 +9,15 @@ It's a lot of work for us to review and accept Pull Requests. Sometimes things c
 ## In-repo tests
 
 A number of tests in this repo are designed to function with local repository
-paths, meaning they can be run directly, and are faster to execute. The `yarn
+paths, meaning they can be run directly, and are faster to execute. The `npm
 test` command is a shortcut to exclude integration tests, eg
 
     bazel test //...
 
 ## Integration tests
 
 In order to test that the rules work outside of this repo, this repo has full bazel WORKSPACEs for e2e tests
- to set up an environment with the package paths matching what end users will see.
+to set up an environment with the package paths matching what end users will see.
 The end-to-end tests in e2e are built this way.
 
 ## Debugging

nodejs/private/node_versions.bzl

@@ -1,5 +1,5 @@
 """ Generated code; do not edit
-Update by running yarn update-nodejs-versions
+Update by running npm run update-nodejs-versions
 """
 
 # @unsorted-dict-items

nodejs/repositories.bzl

@@ -123,7 +123,7 @@ def _download_node(repository_ctx):
     ))
 
 def _prepare_node(repository_ctx):
-    """Sets up BUILD files and shell wrappers for the versions of Node.js, npm & yarn just set up.
+    """Sets up BUILD files and shell wrappers for the versions of Node.js, npm just set up.
 
     Windows and other OSes set up the node runtime with different names and paths, which we hide away via
     the BUILD file here.
@@ -178,7 +178,7 @@ exec "$SCRIPT_DIR/{node}" "$@"
             node = node_bin_relative,
         ))
     else:
-        # Sets PATH for node, npm & yarn and run user script
+        # Sets PATH for node, npm and run user script
         repository_ctx.file("bin/node.cmd", content = """
 @echo off
 SET SCRIPT_DIR=%~dp0
@@ -189,7 +189,7 @@ CALL "%SCRIPT_DIR%\\{node}" %*
     # The entry points for npm for osx/linux and windows
     # Runs npm using appropriate node entry point
     # --scripts-prepend-node-path is set to false since the correct paths
-    # for the Bazel entry points of node, npm & yarn are set in the node
+    # for the Bazel entry points of node, npm are set in the node
     # entry point
     for kind in [
         {"name": "npm", "script": npm_script_relative},

package.json

@@ -1,42 +1,31 @@
 {
-    "name": "@bazel/javascript",
-    "description": "Build JavaScript with Bazel",
-    "private": true,
-    "version": "0.0.0",
-    "keywords": [
-        "javascript",
-        "bazel"
-    ],
-    "homepage": "https://github.com/bazel-contrib/rules_nodejs",
-    "repository": "https://github.com/bazel-contrib/rules_nodejs.git",
-    "license": "Apache-2.0",
-    "engines": {
-        "node": ">=12 <=18",
-        "yarn": ">=1.13.0"
-    },
-    "devDependencies": {
-        "@bazel/bazelisk": "^1.10.1",
-        "@bazel/buildifier": "^3.2.0",
-        "@bazel/buildozer": "^3.2.0",
-        "@commitlint/cli": "^12.1.4",
-        "@commitlint/config-conventional": "^12.1.4",
-        "@types/node": "^15.12.2",
-        "clang-format": "1.5.0",
-        "conventional-changelog-cli": "^2.0.21",
-        "husky": "1.3.1"
-    },
-    "scripts": {
-        "// See comment in .bazelrc": "",
-        "update-codeowners": "./scripts/update_codeowners.sh",
-        "update-nodejs-versions": "node ./scripts/update-nodejs-versions.js > nodejs/private/node_versions.bzl",
-        "format": "git-clang-format",
-        "format-all": "clang-format --glob='{e2e/**/,packages/**/}*.{js,ts}' -i",
-        "update-changelog": "conventional-changelog -p angular -i CHANGELOG.md -s"
-    },
-    "husky": {
-        "hooks": {
-            "pre-push": "check-clang-format \"yarn format\"",
-            "commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
-        }
-    }
+  "name": "@bazel/javascript",
+  "description": "Build JavaScript with Bazel",
+  "private": true,
+  "version": "0.0.0",
+  "keywords": [
+    "javascript",
+    "bazel"
+  ],
+  "homepage": "https://github.com/bazel-contrib/rules_nodejs",
+  "repository": "https://github.com/bazel-contrib/rules_nodejs.git",
+  "license": "Apache-2.0",
+  "engines": {
+    "node": ">=12 <=18"
+  },
+  "packageManager": "pnpm@8.11.0",
+  "devDependencies": {
+    "@bazel/bazelisk": "^1.10.1",
+    "@bazel/buildifier": "^3.2.0",
+    "@bazel/buildozer": "^3.2.0",
+    "@commitlint/cli": "^12.1.4",
+    "@commitlint/config-conventional": "^12.1.4",
+    "@types/node": "^15.12.2",
+    "conventional-changelog-cli": "^2.0.21"
+  },
+  "scripts": {
+    "// See comment in .bazelrc": "",
+    "update-nodejs-versions": "node ./scripts/update-nodejs-versions.js > nodejs/private/node_versions.bzl",
+    "update-changelog": "conventional-changelog -p angular -i CHANGELOG.md -s"
+  }
 }