-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Update outdated documentation link #8205
Update outdated documentation link #8205
Conversation
|
@Naeim-Zarei: This issue is currently awaiting triage. If Ingress contributors determines this is a relevant issue, they will accept it by applying the The 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. |
Welcome @Naeim-Zarei! |
Hi @Naeim-Zarei. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
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. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Naeim-Zarei The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @Naeim-Zarei , Thanks for your contribution. Helps. Can you please rebase and double check your changes. I think @tacf changed that URL in his PR #8203 and it has merged to main. It looks like ccaf572 changed the URL already |
Hi @longwuyuan, Thanks for letting me know! What do you think is the best course of action in this scenario? |
i want to get confirmation from you that what I am guessing is true. can you rebase and check if url has changed to what is the correct url. |
It appears your guess is right. The documentation link has been updated and appears to be working as expected. Should I go ahead and delete this pull request? |
If there is no other change you are proposing, then yes please close this
pr. Welcome any other improvements you want to make. Thank you very much
for the contribution.
Thanks,
; Long
…On Mon, 31 Jan, 2022, 9:01 PM Naeim Zarei, ***@***.***> wrote:
It appears your guess is right. The documentation link has been updated
and appears to be working as expected. Should I go ahead and delete this
pull request?
—
Reply to this email directly, view it on GitHub
<#8205 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGZVWRS4VHTYW2YH2IO53DUY2TNLANCNFSM5NBNVX3A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
What this PR does / why we need it:
The purpose of this pull request is to update an outdated documentation link as per #8192.
Types of changes
Which issue/s this PR fixes
fixes #8192
How Has This Been Tested?
Checklist: