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 date-fns to v2.0.1 #196

Merged
merged 1 commit into from Oct 7, 2019
Merged

upgrade date-fns to v2.0.1 #196

merged 1 commit into from Oct 7, 2019

Conversation

bengry
Copy link
Contributor

@bengry bengry commented Sep 3, 2019

date-fns@2 has been released, and since concurrently is pinned on ^1.x.x, it makes it hard for users concurrently to use date-fns@2 simultaneously, due to conflicting packages.

This PR updates this dependency.

Note that this is a breaking change in concurrently for users who may have passed an invalid format (which previously worked) due to a breaking change in date-fns@2.0.0, to better conform the unicode standard

@coveralls
Copy link

coveralls commented Sep 3, 2019

Pull Request Test Coverage Report for Build 352

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.178%

Totals Coverage Status
Change from base Build 350: 0.0%
Covered Lines: 239
Relevant Lines: 239

💛 - Coveralls

Copy link
Member

@gustavohenke gustavohenke left a comment

Choose a reason for hiding this comment

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

Thanks for this!
Looks good to me with the exception of the Table of Contents changes.

I guess this means a major version for concurrently too?

README.md Outdated Show resolved Hide resolved
@bengry
Copy link
Contributor Author

bengry commented Sep 6, 2019

@gustavohenke technically yes, though it should only users of the progrematic API, and even then - only those who override the default timestamp format.

@maraisr
Copy link

maraisr commented Sep 13, 2019

Super keen on this merge - the only thing holding our code base back...

@bengry
Copy link
Contributor Author

bengry commented Oct 1, 2019

@gustavohenke / @kimmobrunfeldt can this PR be merged, or is it waiting for something? It was approved almost a month ago.

@gustavohenke gustavohenke merged commit 40e7647 into open-cli-tools:master Oct 7, 2019
@gustavohenke
Copy link
Member

Hey everyone! Sorry for the delay. I was waiting for the approval to turn 1 month old 🎂

Jokes apart... you may start using this on v5.0.0 🚢

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

4 participants