Update unicode to v5.0.1#1031
Conversation
| , "maybe" | ||
| , "partial" | ||
| , "prelude" | ||
| , "quickcheck" |
There was a problem hiding this comment.
There were quite a few dependencies added between v5.0.0 and v5.0.1.
Not sure which ones we should be including here (e.g., do we include quickcheck even if it's just a test dependency?)
There was a problem hiding this comment.
We only need to include dependencies in the source code, so nothing that's only a test dependency. I'm assuming that the new dependencies are added just because they were previously transitive?
There was a problem hiding this comment.
Yea, purescript-contrib/purescript-unicode#35 looks like it was dealing with transitive deps. But we don't need any test-only dependencies included in the package sets.
There was a problem hiding this comment.
Alright, reverted the changes to the dependency list 👍🏻
I can't wait for Spago targets 😭
This reverts commit 86fcffd.
2 participants
This PR updates
unicodeto v5.0.1.purescript-contrib/purescript-unicode@v5.0.0...v5.0.1