rules_nodejs icon indicating copy to clipboard operation
rules_nodejs copied to clipboard

Fix repo mapping for bzlmod detection

Open Mivr opened this issue 4 months ago • 1 comments

PR Checklist

  • [ ] Tests for the changes have been added (for bug fixes / features)
  • [x] Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • [x] Bugfix
  • [ ] Feature (please, look at the "Scope of the project" section in the README.md file)
  • [ ] Code style update (formatting, local variables)
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes
  • [ ] CI related changes
  • [ ] Documentation content changes
  • [ ] Other... Please describe:

What is the current behavior?

@bazel/runfiles does not work with non-bzlmod when a _repo_mapping file exists

Issue Number: #3797

What is the new behavior?

The parsing code handles old and new repo_mapping syntax

Does this PR introduce a breaking change?

  • [ ] Yes
  • [ x] No

Mivr avatar Aug 13 '25 17:08 Mivr

Test

All tests were cache hits

2 tests (100.0%) were fully cached saving 53ms.

aspect-workflows[bot] avatar Aug 13 '25 17:08 aspect-workflows[bot]