Skip to content

Run buildifier on the build files and add it to "make check".#2088

Merged
istio-testing merged 1 commit intoistio:masterfrom
PiotrSikora:make_check_buildifier
Jan 24, 2019
Merged

Run buildifier on the build files and add it to "make check".#2088
istio-testing merged 1 commit intoistio:masterfrom
PiotrSikora:make_check_buildifier

Conversation

@PiotrSikora
Copy link
Contributor

Signed-off-by: Piotr Sikora piotrsikora@google.com

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora PiotrSikora requested review from lizan and qiwzhang January 23, 2019 23:48
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jan 23, 2019
Copy link
Contributor

@lizan lizan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copy link
Member

@JimmyCYJ JimmyCYJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

/approve

@istio-testing
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JimmyCYJ, lizan, PiotrSikora

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing istio-testing merged commit 54d5807 into istio:master Jan 24, 2019
@JimmyCYJ
Copy link
Member

@PiotrSikora
Copy link
Contributor Author

@JimmyCYJ this failure is unrelated to this PR (since it's literally a whitespaces change), but it got triggered by changes to WORKSPACE, which in turn invalidated build cache, see #2090.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants