-
Notifications
You must be signed in to change notification settings - Fork 19
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
Site health improvements - january report #199
base: main
Are you sure you want to change the base?
Site health improvements - january report #199
Conversation
fa1b2f3
to
1d4df51
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple minor things but looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@0xlukem there's a merge conflict on one of the automated pages. So what I would do is just accept one or the other, but then you'll need to run the script again to make sure you've pulled in all of the latest updates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@0xlukem actually just saw this PR: #210 What happened was that Martin updated #210 based on the changes made in this mkdocs PR, which hasn't been merged yet: papermoonio/wormhole-mkdocs#129 So your PR will need to be updated again with the new chain config added in that PR. Sorry about the back and forth on this! |
@0xlukem just merged the script changes into |
Description
Based on the January SEMrush report, I updated the
Wormhole-Docs
site health by addressing several broken links and some missing ones.Checklist
wormhole-mkdocs
repo