-
Notifications
You must be signed in to change notification settings - Fork 91
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
Support footnotes in Markdown #324
base: main
Are you sure you want to change the base?
Conversation
From my test on the main branch, the CI has failed already without this change. |
Yes, looks like a recent update to docutils or pygments may have changed the output - would you want to make a separate PR that only updates the test outputs based on current dependencies? |
It was Pygments, when I pinned it to v2.18.0, tests passed. maybe we could pin it to 2.18.0 and create a ticket to track and fix it in a future pull request.
sure, will do. |
I’ve opened a new PR (#325) to update the test outputs, and all tests are now passing. |
32db965
to
87cbbdd
Compare
The test failure has been fixed in 87cbbdd and is ready for review @miketheman |
closes #323