Merge pull request #284 from lume/update-to-yarn-4.0

infra: update to Yarn 4.0

.github/workflows/tests.yml

@@ -28,6 +28,6 @@ jobs:
               run: |
                   npm run fresh
                   npm run build:all
-                  npm run test:all:ci
+                  npm run test:all
               env:
                   CI: true

.yarnrc.yml

@@ -1,6 +1,29 @@
-checksumBehavior: ignore
-nmHoistingLimits: workspaces
+progressBarStyle: patrick # "patrick" | "simba" | "jack" | "hogsfather" | "default",
+
+# without this, Windows was failing to pass checksum validation.
+# checksumBehavior: ignore
+
+# Tells Yarn to install packages to `node_modules`. Without this, it uses its
+# more magical .pnp files, which change the Node runtime to find packages in a
+# different way outside of node_modules, and this does not work for our
+# client-side projects and import maps that we've set up to find packages in
+# node_modules. It would be too cubersome to have the import maps map to .yarn/
+# cache locations which include ugly hashes in their name.
 nodeLinker: node-modules
-plugins:
-    - path: .yarn/plugins/@yarnpkg/plugin-workspace-tools.cjs
-      spec: '@yarnpkg/plugin-workspace-tools'
+
+# Hoist dependencies up to workspace boundaries, rather than all the way up to
+# the root. Without this, importmaps in a client-side examples would not be able
+# to access dependencies because node_modules would not exist in the local
+# project.
+nmHoistingLimits: workspaces
+
+# Yarn recommends enabling this with Yarn 4.0.0 to catch version constraint
+# errors locally before running in CI. In particular, when enabled, Yarn
+# enforces that no two workspaces depend on different versions of any given
+# dependencies, unless explicitly allowed.
+# (https://yarnpkg.com/blog/release/4.0)
+enableConstraintsChecks: true
+
+# Define whether to reuse most common dependency ranges or not when adding dependencies to a package.
+# If true, yarn add will attempt to reuse the most common dependency range in other workspaces.
+preferReuse: true

lerna.json

@@ -1,4 +1,5 @@
 {
+	"packages": [".", "apps/*", "packages/*"],
 	"$schema": "node_modules/lerna/schemas/lerna-schema.json",
 	"version": "independent"
 }

package.json

@@ -61,9 +61,7 @@
 		"build:all:noFail": "npm run clean:all && corepack yarn lerna run build --ignore classy-solid -- -- --noFail",
 		"dev:all": "echo TODO run everything in dev mode with rebuild on changed files",
 		"clean:all": "corepack yarn lerna run clean --ignore classy-solid || echo 'clean:all: missing executables, skipping.'",
-		"test:all": "corepack yarn lerna run test --scope '{lume,@lume/element,element-behaviors,@lume/variable,@lume/eventful,@lume/autolayout,@lume/kiwi,@lume/three-projected-material,custom-attributes,james-bond,lowclass}'",
-		"test:all:ci": "npm run test:all -- --concurrency 1",
-		"//                                      ^ Concurrency needs to be 1 in CI for some reason, but I forgot to comment why.": "",
+		"test:all": "corepack yarn workspaces foreach -Rtp --from 'lume' run test",
 		"// WEBSITE XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX": "",
 		"website": "(npm run docs:serve & p1=$!; npm run website:serve & p2=$!; wait $p1 $p2)",
 		"website:serve": "cd apps/website && meteor",
@@ -186,5 +184,5 @@
 		"web-component",
 		"webcomponent"
 	],
-	"packageManager": "yarn@3.6.4"
+	"packageManager": "yarn@4.0.0"
 }