Skip to content

contrib guide: broken links and release versioning #505

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

Merged
merged 4 commits into from
May 21, 2025

Conversation

miacycle
Copy link
Contributor

@miacycle miacycle commented Apr 25, 2025

Signed-off-by: miacycle 184569369+miacycle@users.noreply.github.com

  1. Fix two broken links
  2. Comment out section on releasing docs / versioning of docs. Not something is currently done.

Signed-off-by: miacycle <184569369+miacycle@users.noreply.github.com>
Copy link

netlify bot commented Apr 25, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 1972d31
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/682de39e0fc22400086aeefb
😎 Deploy Preview https://deploy-preview-505--bejewelled-pegasus-b0ce81.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.

@miacycle miacycle requested review from zihanKuang and leecalcote and removed request for zihanKuang April 25, 2025 21:39
Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

lgtm

@@ -257,7 +257,7 @@ The versioned sites follow this convention:
* `vXXX-YYY.docs.layer5.io` points to the release at vXXX.YYY-branch

We also hook up each version to the dropdown on the website menu bar.
For information on how to update the website to a new version, see the [Layer5 release guide](https://github.com/layer5io/docs/blob/master/docs_dev/releasing.md#releasing-a-new-version-of-the-website).
For information on how to update the website to a new version, see the [Layer5 release guide](https://github.com/layer5io/docs/blob/master/docs_dev/releasing.md#releasing-a-new-version-of-the-website). -->
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if this --> gets properly interpreted or not.

Co-authored-by: Zihan Kuang <127078886+zihanKuang@users.noreply.github.com>
Signed-off-by: Mia Grenell <184569369+miacycle@users.noreply.github.com>
@leecalcote
Copy link
Member

Merge conflicts

@leecalcote
Copy link
Member

@zihanKuang will you take over this PR?

@miacycle
Copy link
Contributor Author

Please!

@hortison
Copy link
Contributor

@zihanKuang, will be working this one?

@zihanKuang
Copy link
Contributor

Yes, I will take over this PR and resolve the merge conflicts.👍

Signed-off-by: Zihan Kuang <zihan_kuang@outlook.com>
Signed-off-by: Zihan Kuang <zihan_kuang@outlook.com>
@zihanKuang zihanKuang merged commit 3059e72 into master May 21, 2025
2 checks passed
@zihanKuang zihanKuang deleted the chore/brokenlinks branch May 21, 2025 14:31
leecalcote pushed a commit that referenced this pull request Jun 10, 2025
contrib guide: broken links and release versioning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants