-
Notifications
You must be signed in to change notification settings - Fork 9
⬆️ Replace mdformat-frontmatter with mdformat-front-matters
#46
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
base: master
Are you sure you want to change the base?
⬆️ Replace mdformat-frontmatter with mdformat-front-matters
#46
Conversation
6dfa0e0 to
f26b25b
Compare
95afeee to
6667793
Compare
c7d33a7 to
a9e2b47
Compare
164faf3 to
bb70265
Compare
Run tests on Python 3.14 and 3.15-dev
Run tests on Python 3.14 and 3.15-dev
bb70265 to
487b64a
Compare
mdformat-frontmatter with mdformat-front-mattersmdformat-frontmatter with mdformat-front-matters
|
All tests are passing, this PR is ready to me merged upstream. |
KyleKing
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks! I can approve, merge, and release once these comments are addressed
| --- | ||
| lastname: Blorothy | ||
| firstname: Dorothy | ||
| lastname: Blorothy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will the change in default to sort front matter be an issue for MyST adoption? If so, I can flip the default for better backward compatibility
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point! We both need input from mdformat-myst maintainers here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There hasn't been any input, so I think we can proceed. I'll communicate this change in the changelog and how to turn it off, then making a breaking release for both that dependency swap and dropping 3.9!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, can you hold off on merging?
I’m going to release a 1.0.0 tonight that doesn’t sort by default. For most, frontmatter is manually written, so better to keep however it was ordered: eyeseast/python-frontmatter#26 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah sure! Take your time! :)
Run tests on Python 3.14 and 3.15-dev
487b64a to
c71ac9b
Compare
KyleKing
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! LGTM!
| --- | ||
| lastname: Blorothy | ||
| firstname: Dorothy | ||
| lastname: Blorothy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There hasn't been any input, so I think we can proceed. I'll communicate this change in the changelog and how to turn it off, then making a breaking release for both that dependency swap and dropping 3.9!
mdformat-frontmatteris not compatible with mdformat 1.0.0, and seems unmaintained.This PR replace it with
mdformat-front-matters, which is actively maintained by @KyleKing and is becoming the de-facto frontmatter plugin in themdformatuniverse, because of its adoption bymdformat-mkdocs.This PR addresses #45.
Also to align the requirements with its dependencies, this PR bump the minimal Python version from 3.9 to 3.10. As a bonus, I activated tests on Python 3.14 and 3.15-dev.
Relates to:
mdformat < 0.8.0requirement butler54/mdformat-frontmatter#37