-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
docs: find broken links using github-action-markdown-link-check #28437
Conversation
Wow, this takes FOREVER. |
@@ -45,7 +45,7 @@ These features are **finished** but currently being tested. They are usable, but | |||
- CACHE_IMPERSONATION | |||
- CONFIRM_DASHBOARD_DIFF | |||
- DRILL_TO_DETAIL | |||
- DYNAMIC_PLUGINS: [(docs)](https://superset.apache.org/docs/configuration/running-on-kubernetes) |
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.
should we add an entry in htaccess? There's one that says RewriteRule ^docs/installation/running-on-kubernetes/$ /docs/installation/kubernetes [R=301,L]
, but could use one for the exact link above too.
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.
Yeah, that makes sense. There are also a whole bunch more bad links that got turned up on this PR... I guess I'll slog through and do the same for those when time allows.
closing in favor of #28424 which is much faster, and more sensible to config. |
SUMMARY
Following up on #28424, using another (relatively deprecated) action, while we figure out if ASF will let us use the first one.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION