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

Fix bad link in governance/ doc #317

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Fix bad link in governance/ doc #317

merged 1 commit into from
Aug 16, 2023

Conversation

kingdonb
Copy link
Member

Joining:

Closing this issue (and the related issues linked above) will enable us to add testing of internal links in the future, to prevent these types of issues on website from regressing.

@kingdonb
Copy link
Member Author

The bad link is currently visible here:

https://deploy-preview-1573--fluxcd.netlify.app/governance/#meetings

the public developer meeting.

Copy link
Member

@makkes makkes left a comment

Choose a reason for hiding this comment

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

Should we also remove the section in this repo's README.md to avoid future divergence?

@kingdonb
Copy link
Member Author

I was getting link blindness, and this was the last of about 150 links I fixed yesterday... I didn't even think to look for it there 😂

Maybe we should just link directly to that, with an absolute link?

/governance/#meetings -> https://github.com/fluxcd/flux2#events

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb
Copy link
Member Author

kingdonb commented Aug 16, 2023

There, that will fix it just as well. No, I do think we should leave the README intact in the community repo

The goal of these updates for the PR was only to fix links with minimal structural changes. If you find any links I've replaced in fluxcd/website#1573 or friends had a more direct analogue, more obvious intended target than whatever I've corrected the links to point at, let's use that instead.

@kingdonb kingdonb merged commit 0ae1ee9 into fluxcd:main Aug 16, 2023
2 checks passed
@kingdonb kingdonb deleted the fix-bad-links branch August 16, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants