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

fix(ref docs): Improve markdown normalization to preserve code block braces #29063

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

charislam
Copy link
Contributor

Braces must be escaped in non-code-blocks (or the site won't build), but preserved in code blocks. This requires a more complex approach than we were using:

  • Parse the initial markdown and identify code blocks
  • Escape braces only in non-code sections of the markdown

Copy link

vercel bot commented Sep 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 3, 2024 5:52pm
5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
design-system ⬜️ Ignored (Inspect) Visit Preview Sep 3, 2024 5:52pm
studio ⬜️ Ignored (Inspect) Visit Preview Sep 3, 2024 5:52pm
studio-self-hosted ⬜️ Ignored (Inspect) Visit Preview Sep 3, 2024 5:52pm
studio-staging ⬜️ Skipped (Inspect) Sep 3, 2024 5:52pm
zone-www-dot-com ⬜️ Skipped (Inspect) Sep 3, 2024 5:52pm

Copy link

supabase bot commented Sep 3, 2024

No changes detected in supabase directory.
This pull request has been ignored for the connected project xguihxuzqibwxjnimxev due to its connection settings.
Go to Project Integrations Settings ↗︎ in order to change this behavior.


Branching Preview Branches by Supabase.
Learn more about Supabase for Git ↗︎.

…braces

Braces must be escaped in non-code-blocks (or the site won't build),
but preserved in code blocks. This requires a more complex approach than
we were using:

- Parse the initial markdown and identify code blocks
- Escape braces only in non-code sections of the markdown
@charislam charislam merged commit f5ec2d2 into master Sep 25, 2024
13 checks passed
@charislam charislam deleted the fix/bracket-escape-refs branch September 25, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants