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

feat: enforce TOC validation in docs build #18974

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ebembi-crdb
Copy link
Contributor

Description

This PR introduces a check to ensure all Markdown files are included in the Table of Contents (TOC). If any Markdown files are unlinked, the documentation build will fail.

Changes

Add check_sidebar.py to validate Markdown file inclusion in the sidebar.
Implement exclusion logic to ignore unnecessary files and directories.

Motivation

To maintain documentation integrity by preventing unlinked content from being published.

Related Issue

This PR resolves DOC-9961.

Copy link

github-actions bot commented Oct 2, 2024

Files changed:

  • src/current/check_sidebar.py

Copy link

netlify bot commented Oct 2, 2024

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit d85442c
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/66fd1a21fa059400082cc1cd

Copy link

netlify bot commented Oct 2, 2024

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit d85442c
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/66fd1a21fc3c1c0008c5c12f

Copy link

netlify bot commented Oct 2, 2024

Netlify Preview

Name Link
🔨 Latest commit d85442c
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/66fd1a21a151d20008bd4701
😎 Deploy Preview https://deploy-preview-18974--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

1 participant