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

Stop skipping lint/format in integration tests #28485

Merged
merged 4 commits into from
Nov 3, 2020

Conversation

howardjohn
Copy link
Member

@howardjohn howardjohn commented Nov 2, 2020

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Pull Request Attributes

Please check any characteristics that apply to this pull request.

[ ] Does not have any changes that may affect Istio users.

@howardjohn howardjohn added the release-notes-none Indicates a PR that does not require release notes. label Nov 2, 2020
@howardjohn howardjohn requested review from a team as code owners November 2, 2020 16:20
@google-cla google-cla bot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Nov 2, 2020
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 2, 2020
@istio-testing istio-testing added the needs-rebase Indicates a PR needs to be rebased before being merged label Nov 2, 2020
@ericvn
Copy link
Contributor

ericvn commented Nov 2, 2020

Not sure if you want to pull the automated common-files update PR: #28492 into this PR or not. You already have some of the common files and both PRs are needing a rebase.

@howardjohn
Copy link
Member Author

Ill pull it in manually

@istio-testing istio-testing removed the needs-rebase Indicates a PR needs to be rebased before being merged label Nov 2, 2020
@howardjohn howardjohn force-pushed the lint/fix-tests branch 2 times, most recently from 8c4768c to 7707b0a Compare November 2, 2020 22:28
@howardjohn
Copy link
Member Author

/retest

@howardjohn
Copy link
Member Author

/retest

2 similar comments
@howardjohn
Copy link
Member Author

/retest

@howardjohn
Copy link
Member Author

/retest

@istio-testing istio-testing merged commit a6b610f into istio:master Nov 3, 2020
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. release-notes-none Indicates a PR that does not require release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants