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.

Sign up for GitHub

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

Jump to bottom

[docs] fix a few 404s #9474

Merged
merged 2 commits into from
Jul 28, 2021
Merged

[docs] fix a few 404s #9474

merged 2 commits into from
Jul 28, 2021

Conversation

polarweasel
Copy link
Contributor

(I've also started testing a link checker, hence the new gitignore entry)

@jaki
Copy link
Contributor

jaki commented Jul 27, 2021

@polarweasel, if you're testing something that only relates to your setup, not anyone else's, consider moving the ignore entry from .gitignore to .git/info/exclude.

@polarweasel
Copy link
Contributor Author

Thanks for the tip, @jaki! I think this is likely to become part of the docs workflow, but I'll take it out for now.

@netlify
Copy link

netlify bot commented Jul 27, 2021

✔️ Deploy Preview for infallible-bardeen-164bc9 ready!

🔨 Explore the source changes: ff3d35c

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/60ff655b6f734a0008247923

😎 Browse the preview: https://deploy-preview-9474--infallible-bardeen-164bc9.netlify.app

@netlify
Copy link

netlify bot commented Jul 27, 2021

✔️ Deploy Preview for infallible-bardeen-164bc9 ready!

🔨 Explore the source changes: 2b6d9f3

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/60ff68e22e57e10007233899

😎 Browse the preview: https://deploy-preview-9474--infallible-bardeen-164bc9.netlify.app

@polarweasel polarweasel requested a review from ddhodge July 27, 2021 20:41
ddhodge
ddhodge approved these changes Jul 27, 2021
View reviewed changes
Copy link
Contributor

@ddhodge ddhodge left a comment

Choose a reason for hiding this comment

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

LGTM

@polarweasel polarweasel merged commit ff06e87 into yugabyte:master Jul 28, 2021
@polarweasel polarweasel deleted the docs-404-from-checker branch July 28, 2021 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ddhodge ddhodge ddhodge approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@polarweasel @jaki @ddhodge