-
Notifications
You must be signed in to change notification settings - Fork 94
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
Upgrade to OCamlformat 0.27.0 #1227
base: master
Are you sure you want to change the base?
Conversation
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.
Great to have a release of ocamlformat with support for 5.2!
Most of the changes are related to the parse-docstrings
being true by default. Making two commits (one setting parse-docstrings=false
and version=0.27.0
and the second one removing parse-docstrings=false
) would make it easier to see what part of the code change.
I left some very minor comments, it looks good to me.
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.
Thanks for the review! I'll split the commits for the final version, for now this is generated by a script.
d87a81d
to
faa2b77
Compare
faa2b77
to
2a50637
Compare
OCamlformat 0.27.0 as been released 🎉 This is ready to be merged. |
The aim of this preview is to gather feedback.
Changelog can be found here: https://github.com/ocaml-ppx/ocamlformat/blob/main/CHANGES.md