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

Allow absolute links in the changelog #49201

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Nov 19, 2024

Disable absolute docs link linting in the changelog. This is because the changelog needs to include absolute docs links in order for our release process to run smoothly, since we need to sync release notes to the changelog.

This change removes the comment that disables the remark messaging linter, since we have removed this custom linter in favor of Vale.

@ptgott
Copy link
Contributor Author

ptgott commented Nov 19, 2024

Depends on this docs PR: gravitational/docs#505

Copy link

🤖 Vercel preview here: https://docs-8a28glvg4-goteleport.vercel.app/docs

@ptgott ptgott force-pushed the paul.gottschling/2024-11-19-changelog-abs-links branch from 4484826 to a59d313 Compare November 19, 2024 16:19
Copy link

🤖 Vercel preview here: https://docs-icvn1cllq-goteleport.vercel.app/docs

@ptgott ptgott force-pushed the paul.gottschling/2024-11-19-changelog-abs-links branch from a59d313 to 4de85d2 Compare November 19, 2024 21:11
Copy link

🤖 Vercel preview here: https://docs-pia0wklhz-goteleport.vercel.app/docs

zmb3

This comment was marked as resolved.

@ptgott ptgott changed the title Add message control directive to the changelog Allow absolute links in the changelog Nov 20, 2024
@ptgott ptgott force-pushed the paul.gottschling/2024-11-19-changelog-abs-links branch from 4de85d2 to 60bebcc Compare November 20, 2024 21:02
This is because the changelog needs to include absolute docs links in
order for our release process to run smoothly, since we need to sync
release notes to the changelog. To do so, add a message control
directive to the changelog.

This change removes the comment that disables the remark messaging
linter, since we have removed this custom linter in favor of Vale.
@ptgott ptgott force-pushed the paul.gottschling/2024-11-19-changelog-abs-links branch from 60bebcc to e465771 Compare November 20, 2024 21:03
Copy link

🤖 Vercel preview here: https://docs-53yndoyvx-goteleport.vercel.app/docs

Copy link

🤖 Vercel preview here: https://docs-17t4mmg3m-goteleport.vercel.app/docs

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from r0mant November 26, 2024 10:08
@ptgott ptgott added this pull request to the merge queue Nov 26, 2024
Merged via the queue into master with commit 82ac161 Nov 26, 2024
40 checks passed
@ptgott ptgott deleted the paul.gottschling/2024-11-19-changelog-abs-links branch November 26, 2024 13:34
@public-teleport-github-review-bot

@ptgott See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants