Skip to content

fix: ignore mdx-like text in federated docs #33

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: ignore mdx-like text in federated docs #33

wants to merge 1 commit into from

Conversation

@lizard-boy
Copy link

@lizard-boy lizard-boy commented Oct 9, 2024

Docs build is breaking because of MDX-like text in federated docs, which shouldn't have any MDX in the first place (they use mkdocs-compatible Markdown). Alter the syntax tree to ignore these tags and display them as plain text, e.g., Nullable<T> should be displayed literally.

Also fixes the redundant header removal to make it more robust (syntax-tree based instead of regex-based.)
@charislam
fix: ignore mdx-like text in federated docs
cb4a90e 
Docs build is breaking because of MDX-like text in federated docs, which shouldn't have any MDX in the first place (they use mkdocs-compatible Markdown). Alter the syntax tree to ignore these tags and display them as plain text, e.g., `Nullable<T>` should be displayed literally. Also fixes the redundant header removal to make it more robust (syntax-tree based instead of regex-based.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@lizard-boy @charislam