-
Notifications
You must be signed in to change notification settings - Fork 88
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
Fix new buildifer warnings #128
Conversation
related bazelbuild/bazel#15025 |
7ce6042
to
2008b5e
Compare
continue | ||
|
||
found = True | ||
for arg in reversed(action.argv): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oquenchil I'm not sure if the order change here is important or not, but there are now different actions and index 1 didn't seem like what we wanted anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, missed this. I thought this PR was just fixing formatting.
I will rewrite all these tests today not to rely on the index. I will get back to you.
Looks like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks a lot! |
They are now part of the bazelbuild/bazel repository. Copies buildifier changes contributed by Keith in #128 RELNOTES:none PiperOrigin-RevId: 435312636 Change-Id: I4d5d2f58d90026c0cdfb5c0a90709de7c28c6ccd
Merged the changes you made here: |
No description provided.