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

Detect # headings with AST #11841

Merged
merged 7 commits into from
Jan 10, 2025
Merged

Detect # headings with AST #11841

merged 7 commits into from
Jan 10, 2025

Conversation

cscheid
Copy link
Collaborator

@cscheid cscheid commented Jan 10, 2025

Adds src/core/lib/markdown-analysis/level-one-headings.ts with a helper function to detect level one headings using the markdown structure of the document instead of a regex.

Closes #11835.

@cscheid cscheid merged commit 06dd45a into main Jan 10, 2025
47 checks passed
@cscheid cscheid deleted the bugfix/issue-11835 branch January 10, 2025 22:11
cderv added a commit that referenced this pull request Jan 14, 2025
Follow up on #11841

Otherwise filter is not found by bundled version (but test passes because dev version works)
@cderv
Copy link
Collaborator

cderv commented Jan 14, 2025

When adding a filter that should be used as-is by Quarto, we need to make sure it is available in the bundled filter files. I opened a PR to fix

@cscheid
Copy link
Collaborator Author

cscheid commented Jan 14, 2025

Ugh, that's right. Thanks for catching that. We really need a way to run some tests on our bundled versions, even if it's just to catch this kind of thing right away.

@cderv
Copy link
Collaborator

cderv commented Jan 14, 2025

I'll let you rebuild a new pre-release later today as I think you may work on some other important fix.

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.

Comments in Code cells are taken as section and increment numbering
2 participants