Skip to content

build: enforce frozen lockfile mode #30969

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 20, 2025

Conversation

alan-agius4
Copy link
Collaborator

This commit adds lockfile_mode=error to the .bazelrc file. This change ensures that any future builds will fail if the lock file is not up-to-date with the BUILD.bazel file, preventing inconsistencies and encouraging developers to commit updated lock files.

This commit adds `lockfile_mode=error` to the `.bazelrc` file. This change ensures that any future builds will fail if the lock file is not up-to-date with the `BUILD.bazel` file, preventing inconsistencies and encouraging developers to commit updated lock files.
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: rc This PR is targeted for the next release-candidate labels Aug 20, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Aug 20, 2025
@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release and removed target: rc This PR is targeted for the next release-candidate labels Aug 20, 2025
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@josephperrott josephperrott added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 20, 2025
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alan-agius4 alan-agius4 merged commit a8b049a into angular:main Aug 20, 2025
35 checks passed
@alan-agius4 alan-agius4 deleted the frozen-lock-bazel branch August 20, 2025 15:55
@alan-agius4
Copy link
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants