Skip to content

Conversation

milesfrain
Copy link
Contributor

@milesfrain milesfrain commented Jan 18, 2021

Description of the change

This PR updates the CHANGELOG in preparation for the upcoming release of this library.

Related: purescript/purescript#3985

@@ -5,12 +5,22 @@ Notable changes to this project are documented in this file. The format is based
## [Unreleased]

Breaking changes:
- Updated code for PureScript 0.14 (#120)
- Removed `fold1Default` and deprecated `foldMap1Default` (#128)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe this was the net result of breaking changes. Would be good to double-check.

Copy link
Member

Choose a reason for hiding this comment

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

I think this is correct

CHANGELOG.md Outdated

New features:
- Added `findMapWithIndex` (#119)
- Added `foldr1` `foldl1` `foldr1Default` `foldl1Default` `foldMap1DefaultR` `foldMap1DefaultL` (#121, #128)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignoring the introduction, then removal of fold1DefaultR fold1DefaultL

CHANGELOG.md Outdated

New features:
- Added `findMapWithIndex` (#119)
- Added `foldr1` `foldl1` `foldr1Default` `foldl1Default` `foldMap1DefaultR` `foldMap1DefaultL` (#121, #128)
Copy link
Member

Choose a reason for hiding this comment

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

Should these be comma-separated, ie.

- Added `foldr1`, `foldl1`, ..., and `foldMap1DefaultL`

CHANGELOG.md Outdated
@@ -5,12 +5,22 @@ Notable changes to this project are documented in this file. The format is based
## [Unreleased]

Breaking changes:
- Updated code for PureScript 0.14 (#120)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's clearer to phrase this as "Added support for PureScript 0.14, and dropped support for all previous versions"

@thomashoneyman thomashoneyman merged commit 56f238e into purescript:master Jan 19, 2021
@milesfrain milesfrain deleted the patch-1 branch January 19, 2021 06:03
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.

3 participants