Skip to content

Conversation

rundel
Copy link
Contributor

@rundel rundel commented Aug 14, 2025

Description

These are just minor fixes to fenced div argument issues I came across in some of the test files.

seismic-monitoring-stations.qmd - missing . from the note class

2024/05/06/9582.qmd - identifier should be before class

2025/03/21/issue-12344.qmd - class should be before key-value

I don't think any of the above are meant to test this pandoc requirement and so should be safe to correct.

Checklist

I don't believe any of the below are applicable in this case.

  • filed a contributor agreement.
  • referenced the GitHub issue this PR closes
  • updated the appropriate changelog in the PR
  • ensured the present test suite passes
  • added new tests
  • created a separate documentation PR in Quarto's website repo and linked it to this PR

@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Aug 14, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@cscheid
Copy link
Collaborator

cscheid commented Aug 19, 2025

Thank you. These look good, and I'm happy this syntax appears to match what we're going to start enforcing more directly with https://github.com/quarto-dev/quarto-markdown

@cscheid cscheid merged commit 4d61e73 into quarto-dev:main Aug 19, 2025
49 of 51 checks passed
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.

3 participants