Skip to content

[9.0.0] Also persist facts in the hidden lockfile#27554

Merged
meteorcloudy merged 1 commit intobazelbuild:release-9.0.0from
bazel-io:cp27522-9.0.0
Nov 6, 2025
Merged

[9.0.0] Also persist facts in the hidden lockfile#27554
meteorcloudy merged 1 commit intobazelbuild:release-9.0.0from
bazel-io:cp27522-9.0.0

Conversation

@bazel-io
Copy link
Copy Markdown
Member

@bazel-io bazel-io commented Nov 5, 2025

Otherwise facts are not added to the default lockfile when an entry in the hidden lockfile for a reproducible extension is used.

Also bump the lockfile version out of caution - users may already have hidden lockfiles that lack facts.

Along the way fix the print behavior of Facts which resulted in Facts(<unknown object net.starlark.java.eval.Dict>) since it called the wrong repr override.

Closes #27522.

PiperOrigin-RevId: 828559479
Change-Id: I77f002fa0e74c23b9fab5bb05948bcbe76f8f49b

Commit bed7a1f

Otherwise facts are not added to the default lockfile when an entry in the hidden lockfile for a reproducible extension is used.

Also bump the lockfile version out of caution - users may already have hidden lockfiles that lack facts.

Along the way fix the `print` behavior of `Facts` which resulted in `Facts(<unknown object net.starlark.java.eval.Dict>)` since it called the wrong `repr` override.

Closes bazelbuild#27522.

PiperOrigin-RevId: 828559479
Change-Id: I77f002fa0e74c23b9fab5bb05948bcbe76f8f49b
@bazel-io bazel-io requested a review from a team as a code owner November 5, 2025 19:55
@bazel-io bazel-io added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Nov 5, 2025
@bazel-io bazel-io requested a review from Wyverald November 5, 2025 19:55
@meteorcloudy meteorcloudy added this pull request to the merge queue Nov 6, 2025
Merged via the queue into bazelbuild:release-9.0.0 with commit 05cba19 Nov 6, 2025
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants