-
Notifications
You must be signed in to change notification settings - Fork 15
Enrich expression API #71
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
Enrich expression API #71
Conversation
|
@arnaudleclerc Could you please review? Thanks. |
Codecov Report
@@ Coverage Diff @@
## develop #71 +/- ##
========================================
Coverage 86.45% 86.45%
========================================
Files 124 124
Lines 3655 3655
========================================
Hits 3160 3160
Misses 495 495
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
@mitikov Thanks for the PR, this is a great improvement! Everything looks fine. Do you think it is possible to add some unit tests on the new code ? |
|
@arnaudleclerc Sure, here you go :) |
|
Awesome, thank you! |
Adding primitive APIs to reduce the volume of boilerplate code to be written while working with expressions.
Inspired by #70