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

Remove 'title' frontmatter and improve title handling logic #196

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Jan 14, 2025

Replaced the 'title' frontmatter across documentation with level 1 headings, ensuring compliance with new markdown conventions. Updated the rendering logic in the codebase to emit warnings for deprecated titles and fallback to file names if no headings are found. These changes simplify title management and enhance markdown compatibility.

Requires #193 to be implemented before we can merge and release this otherwise we create to much noise for the bug bash tomorrow.

Replaced the 'title' frontmatter across documentation with level 1 headings, ensuring compliance with new markdown conventions. Updated the rendering logic in the codebase to emit warnings for deprecated titles and fallback to file names if no headings are found. These changes simplify title management and enhance markdown compatibility.
@Mpdreamz Mpdreamz linked an issue Jan 14, 2025 that may be closed by this pull request
Added the `--strict` flag to the docs builder for stricter validation. Refactored title handling logic to delay fallback to filename when no title is found. Minor fixes in generated link text in docs for better clarity.
@Mpdreamz
Copy link
Member Author

Updated this to ensure:

  • Warns on yaml front matter titles but they are still used as fallback titles for the documents.

@Mpdreamz Mpdreamz added authoring Relates to our markdown parser feature labels Jan 15, 2025
@Mpdreamz Mpdreamz requested a review from reakaleek January 15, 2025 10:42
@bmorelli25
Copy link
Member

bmorelli25 commented Jan 19, 2025

#193 has been fixed. This PR can now be merged!

@bmorelli25
Copy link
Member

bmorelli25 commented Jan 20, 2025

Tested this branch with the integration-docs.md docset and I'm seeing two titles on every page:

Screenshot 2025-01-19 at 8 34 46 PM Screenshot 2025-01-19 at 8 35 41 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authoring Relates to our markdown parser feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch from title frontmatter to H1
3 participants