Add bazel mod tidy dry run#25246
Conversation
Release Notes:
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
* Allow bazel mod tidy to take a `write` option that determines whether fixes are made to files in place -> Defaults to true
github-actions
bot
added
awaiting-review
numine777
force-pushed
the
add-mody-tidy-nowrite
branch
from
1f08d9e to
de8880d
Compare
src/main/java/com/google/devtools/build/lib/bazel/bzlmod/modcommand/ModOptions.java
Show resolved
Hide resolved
JIRA Tracking ID: Description: Test Notes: Release Notes:
JIRA Tracking ID: Description: Test Notes: Release Notes:
|
Please send a PR to the main branch |
github-actions
bot
removed
the
awaiting-review
iancha1992
added
the
potential 7.x cherry-picks
|
@numine777 If this is merged to the master branch, then please feel free to cherry-pick the commit to our new |
|
@fmeum Looks like we're out of time and we're about to get the rc1 out soon for 7.6.0. Are you okay with not including this for our 7.6.0 release? |
|
I apologize for missing the deadline. What would you like me to target for my next inclusion for the bazel 7 branch? |
|
@numine777 Please create a PR to the |
iancha1992
removed
the
potential 7.x cherry-picks
4 participants
This PR addresses issue 24263
We needed this change for our CI as we are migrating to bzlmod. I have patched our current internal version (7.3.1) with this change and plan to update the patch when we move to Bazel 8 (at which time I can submit a PR to future Bazel 8 releases as well).
Please let me know if this PR needs any additional updates to documentation or testing before it is viable. I merely saw that the issue was tagged as a good first issue and thought I would jump in!