Skip to content

Conversation

@jbedard
Copy link
Member

@jbedard jbedard commented Nov 7, 2025

Switching our tests to pnpm v9 also changed the default version for rules_js users who do not manually set the version.

This issue was raised in #2438

Changes are visible to end-users: no

Test plan

  • Covered by existing test cases
  • New test cases added

@aspect-workflows
Copy link

aspect-workflows bot commented Nov 7, 2025

Bazel 7 (Test)

Waiting for runner...


Bazel 8 (Test)

Waiting for runner...


Bazel 7 (Test)

e2e/bzlmod

Waiting for runner...


Bazel 7 (Test)

e2e/git_dep_metadata

Waiting for runner...


Bazel 7 (Test)

e2e/gyp_no_install_script

Waiting for runner...


Bazel 7 (Test)

e2e/js_image_oci

Waiting for runner...


Bazel 7 (Test)

e2e/npm_link_package

Waiting for runner...


Bazel 7 (Test)

e2e/npm_link_package-esm

Waiting for runner...


Bazel 7 (Test)

e2e/npm_link_package-rerooted

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_disable_hooks

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_empty

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_exclude_package_contents

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_multi

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_partial_clone

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_replace_packages

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_lock_subdir_patch

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_package_lock

Waiting for runner...


Bazel 7 (Test)

e2e/npm_translate_yarn_lock

Waiting for runner...


Bazel 7 (Test)

e2e/package_json_module

Waiting for runner...


Bazel 7 (Test)

e2e/patch_from_repo

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_lockfiles

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_repo_install

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_version

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_workspace

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_workspace_deps

Waiting for runner...


Bazel 7 (Test)

e2e/pnpm_workspace_rerooted

Waiting for runner...


Bazel 7 (Test)

e2e/repo_mapping

Waiting for runner...


Bazel 7 (Test)

e2e/rules_foo

Waiting for runner...


Bazel 7 (Test)

e2e/runfiles

Waiting for runner...


Bazel 7 (Test)

e2e/stamped_package_json

Waiting for runner...


Bazel 7 (Test)

e2e/vendored_node

Waiting for runner...


Bazel 7 (Test)

e2e/vendored_tarfile

Waiting for runner...


Bazel 7 (Test)

e2e/verify_patches

Waiting for runner...


Bazel 7 (Test)

e2e/worker

Waiting for runner...


Bazel 7 (Test)

e2e/workspace

Waiting for runner...


Buildifier      Format

@jbedard jbedard force-pushed the pnpm-version-check branch from 21a66f0 to ce1d093 Compare November 7, 2025 20:28
@jbedard jbedard merged commit f4494f2 into main Nov 7, 2025
223 of 225 checks passed
@jbedard jbedard deleted the pnpm-version-check branch November 7, 2025 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants