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

Upgrade Ormolu to 0.0.5.0 #1078

Merged
merged 5 commits into from
Apr 27, 2020
Merged

Upgrade Ormolu to 0.0.5.0 #1078

merged 5 commits into from
Apr 27, 2020

Conversation

mheinzel
Copy link
Contributor

@mheinzel mheinzel commented Apr 25, 2020

Changelog

Most changes are caused by:

  • Records with a single data constructor are now formatted more compactly. Issue 425.

  • Added experimental support for simple CPP. Issue 415.

Other notable changes we can start making use of:

  • Grouping of statements in do-blocks is now preserved. Issue 74.

  • Comments on pragmas are now preserved. Issue 216.

Copy link
Member

@akshaymankar akshaymankar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelog entry for most of the diff:

I haven't run it locally, but looks good 👍

Can we do 2 quick releases with and without this change so it is easier to look at release PRs.

@mheinzel
Copy link
Contributor Author

I think it's not that one, but yes, good point to mention why we have these diffs. I updated the PR description.

@mheinzel mheinzel merged commit f80faf3 into develop Apr 27, 2020
@mheinzel mheinzel deleted the mheinzel/update-ormolu branch April 27, 2020 11:09
@akshaymankar akshaymankar mentioned this pull request May 7, 2020
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.

None yet

2 participants