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: update base domain to kubefirst.konstruct.io #803

Merged
merged 4 commits into from
Oct 22, 2024
Merged

Conversation

muse-sisay
Copy link
Contributor

Description

PR updates docs.kubefirst.io to `kubefirst.konstruct.io/docs'

Related Issue(s)

NA

How to test

npm start

@fharper
Copy link
Contributor

fharper commented Oct 22, 2024

@muse-sisay can you fix the branding issue (the failed workflow)?

Vale matched the letters `konstruct` in the middle of another sentence. Eg it would match it on `https://
kubefirst.konstruct.io/docs` and give an error. This is an acceptable scenario. This commit add regex match for konstruct
@muse-sisay
Copy link
Contributor Author

Hey @fharper,
I updated the substitution(swap) rule to use regex to match konstruct -> ^konstruct$.

@fharper
Copy link
Contributor

fharper commented Oct 22, 2024

@muse-sisay: I understand why you did that, but I didn't want a regex for the reason that we should not link on link. There should be text for the link for a better experience, so that rule is sometimes showing us mistake like this one. Let me revert and fix the README :)

@fharper fharper merged commit 8d37894 into main Oct 22, 2024
13 checks passed
@fharper fharper deleted the konstruct-links branch October 22, 2024 17:44
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