-
Notifications
You must be signed in to change notification settings - Fork 16
Update changelog since v5.1.0 #45
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
Conversation
CHANGELOG.md
Outdated
|
|
||
| Breaking changes: | ||
| - Added support for PureScript 0.14 and dropped support for all previous versions (#37, #43) | ||
| - Added `foldr1` and `foldl` implementations to `Foldable1 (Tuple a)` instance and removed `foldl1` (#39, #43) |
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.
I don't think this is a breaking change.
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.
Move it down to 'Other Improvements' similar to what Miles did in another PR?
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.
I wasn't sure where to put this; it's changed because the Foldable1 class has changed, and if we were to upgrade the dependency and apply this change in the library in normal circumstances it would be breaking. (For example, you could no longer use fold1 via the class.)
But since it's part of adding support for 0.14, perhaps "other improvements" is the way to go for this particular case.
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.
I've moved it to "other improvements" for the time being.
| - Added support for PureScript 0.14 and dropped support for all previous versions (#37, #43) | ||
| - Added `foldr1` and `foldl` implementations to `Foldable1 (Tuple a)` instance and removed `foldl1` (#39, #43) | ||
|
|
||
| New features: |
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.
JordanMartinez
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.
A few fixes
Description of the change
This PR updates the CHANGELOG in preparation for the upcoming release of this library.
Related: purescript/purescript#3985