Skip to content

Fix lint in the announcement#5106

Merged
frankbu merged 4 commits intomasterfrom
lin-test-2
Oct 9, 2019
Merged

Fix lint in the announcement#5106
frankbu merged 4 commits intomasterfrom
lin-test-2

Conversation

@linsun
Copy link
Copy Markdown
Member

@linsun linsun commented Oct 8, 2019

Please provide a description for what this PR is for.

And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.

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

@linsun linsun requested a review from a team as a code owner October 8, 2019 18:19
@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 Oct 8, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 8, 2019
@geeknoid
Copy link
Copy Markdown
Contributor

geeknoid commented Oct 8, 2019

Rebasing will get rid of the bad link error in the lint job.

@linsun
Copy link
Copy Markdown
Member Author

linsun commented Oct 8, 2019

@geeknoid u mean wait till #5103 is merged?

@geeknoid
Copy link
Copy Markdown
Contributor

geeknoid commented Oct 8, 2019

@linsun I thought #5103 has been merged already. Anyway, it's merged now.

You still have a broken link though:

@istio-testing istio-testing added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 9, 2019
@istio-testing
Copy link
Copy Markdown
Contributor

@linsun: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
lint_istio.io 71c958d link /test lint_istio.io
Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@frankbu frankbu merged commit f556d09 into master Oct 9, 2019
@frankbu frankbu deleted the lin-test-2 branch October 9, 2019 16:16
@istio-testing
Copy link
Copy Markdown
Contributor

In response to a cherrypick label: new pull request created: #5114

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. 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.

6 participants