-
Notifications
You must be signed in to change notification settings - Fork 402
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
Buildifier is not being enforced everywhere #611
Comments
It seems buildifier doesn't enter directories that are actually local repositories, which all raze repos are. For the rest, I'm still looking. |
Ah I was wrong, https://github.com/bazelbuild/continuous-integration/blob/master/buildkite/README.md#running-buildifier-on-ci says the buildifier only looks into |
I'm asking the Bazel CI folks. |
This has been fixed as far as I know. |
There are a lot of things out of date when you run
buildifier -lint=fix -mode=fix -warnings=all -r ./
. Many things can be fixed by re-runningcargo-raze
on the latest version, but there are a few other files that have changes that probably shouldn't. Namely things in./rust
and./test
:The text was updated successfully, but these errors were encountered: