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

Broken link need to fix #6868

Closed
amolmote opened this issue Sep 10, 2022 · 14 comments · Fixed by #6870
Closed

Broken link need to fix #6868

amolmote opened this issue Sep 10, 2022 · 14 comments · Fixed by #6870
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one.

Comments

@amolmote
Copy link
Member

amolmote commented Sep 10, 2022

This is a Bug Report

Problem:
visit this page pull-requests
when we click on this Kubernetes Enhancement Proposal process it returns 404 not found page,so this needs to be fix.

Proposed Solution:
correct link of md file require to provide.

Page to Update:
https://www.kubernetes.dev/docs/guide/pull-requests/

/sig docs
/assign
/kind bug

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Sep 10, 2022
@k8s-ci-robot
Copy link
Contributor

@amolmote: The label(s) language/en cannot be applied, because the repository doesn't have them.

In response to this:

This is a Bug Report

Problem:
visit this page pull-requests
when we click on this Kubernetes Enhancement Proposal process it returns 404 not found page,so this needs to be fix.

Proposed Solution:
correct link of md file require to provide.

Page to Update:
https://www.kubernetes.dev/docs/guide/pull-requests/

/language en
/assign
/kind bug

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.

@k8s-ci-robot
Copy link
Contributor

@amolmote: There are no sig labels on this issue. Please add an appropriate label by using one of the following commands:

  • /sig <group-name>
  • /wg <group-name>
  • /committee <group-name>

Please see the group list for a listing of the SIGs, working groups, and committees available.

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.

@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Sep 10, 2022
@souravsk
Copy link

The link is working fine.Can you recheck it.

@amolmote
Copy link
Member Author

This link is working fine which I provided in issue but link of KEP is not working properly

@amolmote
Copy link
Member Author

I have checked it multiple times then I have raised this issue

@souravsk
Copy link

It's not working for the desktop Brower previously when I checked it was working it opened in the GitHub mobile application.

@souravsk
Copy link

i'm working on this

@amolmote
Copy link
Member Author

I have assigned it to myself already

@souravsk
Copy link

okay let me if need any help

@amolmote
Copy link
Member Author

yes sure I am waiting for the sig, once it is approved by any member i will the pr.

@souravsk
Copy link

okay

@souravsk
Copy link

can you give me the file location of page

@dims
Copy link
Member

dims commented Sep 10, 2022

https://github.com/kubernetes/enhancements/blob/master/keps/0001-kubernetes-enhancement-proposal-process.md should be replaced with
https://github.com/kubernetes/enhancements/blob/master/keps/sig-architecture/0000-kep-process/README.md

@amolmote
Copy link
Member Author

yes will do this changes thank you @dims

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants