Add a dry-run option to bazel mod tidy
This PR addresses issue https://github.com/bazelbuild/bazel/issues/24263
We needed this change for our CI as we are migrating to bzlmod. This PR now addresses the master branch to make the patch as useful as possible.
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!
+1 to get this landed
I sent https://github.com/bazelbuild/buildtools/pull/1373 to make it easier to implement this feature. Let's wait for that to land as well as a release, then our job here will become much easier.
Based on that change, I was able to simplify the current PR: https://github.com/bazelbuild/bazel/compare/master...fmeum:bazel:fork/numine777/feature/add-mody-tidy-nowrite?expand=1
@fmeum Thanks much! I apologize for letting this get stale. My team pivoted back to using WORKSPACE for the time being, so the demand hasn't been as strong on our end
Don't worry, thanks for kicking this off!
@bazel-io fork 7.7.0
Any update on getting this landed?
This needs https://github.com/bazelbuild/buildtools/pull/1373, which has been rolled back