locking: enforce Bzlmod lockfile consistency #69
locking: enforce Bzlmod lockfile consistency #69dcalavrezo-qorix wants to merge 4 commits intomainfrom
Conversation
- add reusable lockfile check workflow - track MODULE.bazel.lock (remove from .gitignore) - enforce --lockfile_mode=error in CI and scripts Addresses eclipse-score/score#2628
don't use it for bazel info. it's a bit excessive Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
ran a lockfile update Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
updated lockfile Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
| requirements_lock = "//tests/test_cases:requirements.txt.lock", | ||
| ) | ||
|
|
||
| use_repo(python) |
There was a problem hiding this comment.
that was the bazel mod tidy does
There was a problem hiding this comment.
@pawelrutkaq @PiotrKorkus if none of the python repos/parts are referenced ( thereare no @python_* bla -bla labels in BUILD/MODULE) it considers it redundant and removes it
There was a problem hiding this comment.
Python tests passed, so it looks like it was not needed indeed.
PiotrKorkus
left a comment
There was a problem hiding this comment.
build_qnx8 doesnt check --lockfile_mode but I think the workflow in cicd-workflows needs to be modified to support extra flag
| requirements_lock = "//tests/test_cases:requirements.txt.lock", | ||
| ) | ||
|
|
||
| use_repo(python) |
There was a problem hiding this comment.
Python tests passed, so it looks like it was not needed indeed.
Notes for Reviewer
Pre-Review Checklist for the PR Author
Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Closes #