-
Notifications
You must be signed in to change notification settings - Fork 544
Add Json.traverse* methods #1648
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
Codecov Report
@@ Coverage Diff @@
## master #1648 +/- ##
==========================================
- Coverage 86.13% 85.03% -1.10%
==========================================
Files 61 61
Lines 2423 2453 +30
Branches 138 145 +7
==========================================
- Hits 2087 2086 -1
- Misses 336 367 +31
Continue to review full report at Codecov.
|
|
Thanks for the PR. Love the idea. What do you think of changing the signature though to be a little more flexible. From: To: I think this would allow more transformation within the monadic context. This would make it more akin to the |
|
I think this would a nice set of combinators to have, @sergeykolbasov are you willing to make the changes suggested by @zarthross ? The boolean case is an example, it would be the same for all the new traverseX functions |
|
mimaErrors are genuine. We do not have these in previous releases, so we need to add exclusions rules here. |
|
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## series/0.14.x #1648 +/- ##
=================================================
- Coverage 85.13% 85.03% -0.10%
=================================================
Files 76 61 -15
Lines 2772 2453 -319
Branches 369 145 -224
=================================================
- Hits 2360 2086 -274
+ Misses 412 367 -45
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
It's similar to
Json.map*transformations but with a drop ofF[_] : ApplicativeWe mangle a lot with JSON AST, often in some monadic context. This collection of methods would come in handy.