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

Correct Vale lint errors in partials #49798

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Dec 4, 2024

One edge case is the structure.architecture-h2 warning in docs/pages/includes/database-access/rds-proxy.mdx. This is technically correct, since the "How it works" section is before the "Prerequisites" section. We can determine whether this is a bug to fix in a separate change.

@ptgott ptgott force-pushed the paul.gottschling/2024-12-04-lint-includes branch from 078d6b7 to c8b66b8 Compare December 9, 2024 19:38
@ptgott ptgott enabled auto-merge December 9, 2024 19:40
One edge case is the `structure.architecture-h2` warning in
`docs/pages/includes/database-access/rds-proxy.mdx`. This is technically
correct, since the "How it works" section is before the "Prerequisites"
section. We can determine whether this is a bug to fix in a separate
change.
@ptgott ptgott force-pushed the paul.gottschling/2024-12-04-lint-includes branch from c8b66b8 to a8f9ad6 Compare December 9, 2024 19:40
Copy link

github-actions bot commented Dec 9, 2024

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

@ptgott ptgott added this pull request to the merge queue Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

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

Merged via the queue into master with commit b975c0e Dec 9, 2024
41 of 42 checks passed
@ptgott ptgott deleted the paul.gottschling/2024-12-04-lint-includes branch December 9, 2024 20:09
@public-teleport-github-review-bot

@ptgott See the table below for backport results.

Branch Result
branch/v15 Failed
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.

4 participants