Skip to content
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

Add format command to force unconditional formatting #1186

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Aug 24, 2023

This allows users of buildozer to format files without also having buildifier installed. This was possible previously by making a trivial edit and reverting it, but a dedicated command is cleaner.

Will be used as part of the upcoming bazel mod tidy command.

This allows users of buildozer to format files without also having
buildifier installed. This was possible previously by making a trivial
edit and reverting it, but a dedicated command is cleaner.

Will be used as part of the upcoming `bazel mod tidy` command.
@fmeum
Copy link
Contributor Author

fmeum commented Aug 24, 2023

@vladmos Could you review? We need this for bazel mod tidy as it should format unconditionally and we don't also want to require users to install buildifier in addition to buildozer.

@vladmos vladmos merged commit b163fcf into bazelbuild:master Aug 25, 2023
@fmeum fmeum deleted the format branch August 25, 2023 12:09
apattidb pushed a commit to databricks/buildtools that referenced this pull request May 10, 2024
This allows users of buildozer to format files without also having
buildifier installed. This was possible previously by making a trivial
edit and reverting it, but a dedicated command is cleaner.

Will be used as part of the upcoming `bazel mod tidy` command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants