bazel icon indicating copy to clipboard operation
bazel copied to clipboard

Add a dry-run option to bazel mod tidy

Open numine777 opened this issue 9 months ago • 1 comments

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!

numine777 avatar Mar 12 '25 21:03 numine777

+1 to get this landed

dws avatar Jun 13 '25 17:06 dws

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 avatar Jun 21 '25 13:06 fmeum

@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

numine777 avatar Jun 21 '25 13:06 numine777

Don't worry, thanks for kicking this off!

fmeum avatar Jun 21 '25 15:06 fmeum

@bazel-io fork 7.7.0

iancha1992 avatar Sep 12 '25 22:09 iancha1992

Any update on getting this landed?

dws avatar Oct 26 '25 18:10 dws

This needs https://github.com/bazelbuild/buildtools/pull/1373, which has been rolled back

fmeum avatar Oct 26 '25 19:10 fmeum