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

Repoint doctest build #13717

Merged
merged 6 commits into from
Jun 23, 2022
Merged

Repoint doctest build #13717

merged 6 commits into from
Jun 23, 2022

Conversation

tcsc
Copy link
Contributor

@tcsc tcsc commented Jun 22, 2022

Updates the image used to run the doc-tests CI build and pulls in changes from #13774 to fix compatibility issues with the new image.

See-Also: #13457
Co-Authored-By: @ptgott

@tcsc tcsc requested a review from ptgott June 22, 2022 01:36
@tcsc tcsc assigned tcsc and ptgott Jun 22, 2022
@ptgott
Copy link
Contributor

ptgott commented Jun 22, 2022

Once we merge this, we'll need to merge this branch as well to prevent master from breaking the linter.

@tcsc
Copy link
Contributor Author

tcsc commented Jun 23, 2022

Once we merge this, we'll need to merge this branch as well to prevent master from breaking the linter.

Best thing is probably to add those commits here, otherwise this will never pass CI

ptgott and others added 3 commits June 23, 2022 11:35
In gravitational/docs, relative links in partials are revaluated
relative to the filepath of the partial, not to the page that includes
that partial. In other words, a link in the same partial will be
resolved differently depending on the filepath of the page that
includes it.

To accommdate this link resolution method, this change edits relative
links within the CHANGELOG to be relative to that file, rather than to
docs/pages/changelog.mdx, which includes it.
@tcsc tcsc marked this pull request as ready for review June 23, 2022 01:39
@github-actions github-actions bot requested review from Joerger and timothyb89 June 23, 2022 01:40
@tcsc tcsc merged commit 93d0898 into master Jun 23, 2022
@tcsc tcsc deleted the tcsc/port-doctests-to-docs-image branch June 23, 2022 04:17
ptgott added a commit that referenced this pull request Jul 5, 2022
* Repoint docttest build

* relabel image

* Fix CHANGELOG links for new resolution method

In gravitational/docs, relative links in partials are revaluated
relative to the filepath of the partial, not to the page that includes
that partial. In other words, a link in the same partial will be
resolved differently depending on the filepath of the page that
includes it.

To accommdate this link resolution method, this change edits relative
links within the CHANGELOG to be relative to that file, rather than to
docs/pages/changelog.mdx, which includes it.
ptgott added a commit that referenced this pull request Jul 5, 2022
* Repoint docttest build

* relabel image

* Fix CHANGELOG links for new resolution method

In gravitational/docs, relative links in partials are revaluated
relative to the filepath of the partial, not to the page that includes
that partial. In other words, a link in the same partial will be
resolved differently depending on the filepath of the page that
includes it.

To accommdate this link resolution method, this change edits relative
links within the CHANGELOG to be relative to that file, rather than to
docs/pages/changelog.mdx, which includes it.
ptgott added a commit that referenced this pull request Jul 7, 2022
* Repoint docttest build

* relabel image

* Fix CHANGELOG links for new resolution method

In gravitational/docs, relative links in partials are revaluated
relative to the filepath of the partial, not to the page that includes
that partial. In other words, a link in the same partial will be
resolved differently depending on the filepath of the page that
includes it.

To accommdate this link resolution method, this change edits relative
links within the CHANGELOG to be relative to that file, rather than to
docs/pages/changelog.mdx, which includes it.
@webvictim webvictim mentioned this pull request Jul 12, 2022
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.

3 participants