-
Notifications
You must be signed in to change notification settings - Fork 289
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
Added Annotate Provide Option to inject named values without relying on Out #656
Conversation
Codecov Report
@@ Coverage Diff @@
## master #656 +/- ##
=========================================
+ Coverage 95.58% 95.7% +0.12%
=========================================
Files 9 9
Lines 385 396 +11
=========================================
+ Hits 368 379 +11
Misses 12 12
Partials 5 5
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, thank you. This is excellent!
I left minor style nits but I see no issues besides that. Can I get another
pair of eyes, @akshayjshah @glibsm @mh-park?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm.
Thank you for taking the time to contribute
All Done. |
Thanks, @srikrsna! |
Picking off where #633 was left out for #610. I believe once this is implemented #653 can be reasoned about in a better way.
I've also tried to incorporate all code review comments from #633 and also added some documentation.