Skip to content

Conversation

@codeEvolveZenith345
Copy link
Contributor

This PR updates all broken "Contribute" links and redirection rules throughout the documentation and config files to point to the official Knative contributor guide. Prevents 404 errors and improves developer onboarding.

Proposed Changes

  • Updates /contributing/ redirects and documentation links to use https://knative.dev/community/contributing/
  • Fixes internal and external navigation for contributors
  • Improves overall developer experience and documentation quality

Fixes #6483

@netlify
Copy link

netlify bot commented Nov 1, 2025

Deploy Preview for knative ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 634b690
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/690e5bf31354600007d90552
😎 Deploy Preview https://deploy-preview-6484--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 1, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@knative-prow knative-prow bot requested review from mmejia02 and skonto November 1, 2025 05:14
@knative-prow
Copy link

knative-prow bot commented Nov 1, 2025

Welcome @codeEvolveZenith345! It looks like this is your first PR to knative/docs 🎉

@knative-prow knative-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 1, 2025
…butor guide. Prevents 404 errors and improves developer documentation and onboarding experience

Signed-off-by: codeEvolveZenith345 <suntiwari3495@gmail.com>
Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the redirects should stay same-site, rather than hard-coding the hostname part of the URL -- this allows us to check out the behavior on previews, like

https://deploy-preview-6484--knative.netlify.app/docs/contributing/about

(Oddly, this redirect still works -- so it may be that something else is overwriting the hostname in the redirect map.)

@evankanderson
Copy link
Member

/assign

Signed-off-by: codeEvolveZenith345 <suntiwari3495@gmail.com>
Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Making the the "use host-relative paths" suggestion as suggested edits.

codeEvolveZenith345 and others added 2 commits November 8, 2025 02:21
Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
Co-authored-by: Evan Anderson <evan.k.anderson@gmail.com>
@codeEvolveZenith345
Copy link
Contributor Author

hey @evankanderson
I am grateful for the review and commit-suggestion, I have implemented the suggestions,
The PR is ready to be merged!

Thank you.

Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 7, 2025
@knative-prow
Copy link

knative-prow bot commented Nov 7, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: codeEvolveZenith345, evankanderson

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 7, 2025
@knative-prow knative-prow bot merged commit 625de5f into knative:main Nov 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken "Contribute" link on Knative documentation site (404 error)

2 participants