-
Notifications
You must be signed in to change notification settings - Fork 831
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(docs): Fix various broken links in docs #5047
fix(docs): Fix various broken links in docs #5047
Conversation
@cliveseldon I think there are a couple of links which aren't just broken, but rather the content has fundamentally changed. For example, see the link right at the end of this section for pipeline triggers. This errors example is also only referenced, but seemingly never used: $ rg 'error\.yaml' docs/source/
docs/source/contents/pipelines/index.md
190:```{literalinclude} ../../../../samples/pipelines/error.yaml |
Linting issues seem to be wholly unrelated to this PR. Any way we can ignore @cliveseldon? These seem to have only cropped up within the last few hours, maybe? Builds from 2 days ago didn't have this, but one from earlier today does. For me locally, with Go 1.19.6 and |
What this PR does / why we need it:
This PR fixes issues in the Core v2 docs from the models page to Rclone details regarding Docker and Kubernetes.
Which issue(s) this PR fixes:
Fixes #5045
Special notes for your reviewer:
The use of
myst_heading_anchors
fixes a number of other broken links due to the default for this being 2.See these warnings from a local build:
The usage metrics page has
### Extra publish URLs
, i.e. an H3, which is not found with the H2 default.