-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
CHANGELOG.md: update outdated link #44949
CHANGELOG.md: update outdated link #44949
Conversation
Hi @zhangxiaoyu-zidif. 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 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. |
Updating 1.6 makes sense, but I don't think we need to go back and update links from 1.4, etc. |
@cblecker I am not sure that if it's worth updating these links. If you think it is unnecessary, that would be fine to close this PR. :). |
This seems fine to me, I agree with @cblecker wrt 1.4. |
@k8s-bot ok to test |
@brendandburns Hi! I do not know what is wrt1.4. Dose it mean I need fix these links from v1.4 on. :) |
@zhangxiaoyu-zidif You've suggested changes around lines 1000-1006 and lines 1210-1213. Those are part of the 1.6 changelog. You've also suggested changes around 4517-4521, and 6243-6250. Those are in the 1.4 and 1.2 changelog sections. The 1.6 changes are fine, but please remove changes in the 1.4 and 1.2 sections. We don't want to change history that far back. |
@cblecker @brendandburns |
@k8s-bot node e2e test this |
@zhangxiaoyu-zidif wrt == "with regards too" apologies, it's an english acronym, I should have been more verbose. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brendandburns, zhangxiaoyu-zidif
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@brendandburns @cblecker Thank you! you are both nice and kind~ :) |
Automatic merge from submit-queue |
I am not sure that if the file was generated automatically.
But I find some links were outdated.
Release note:
NONE