-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Fix param value bug #1467
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
Fix param value bug #1467
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1467 +/- ##
==========================================
+ Coverage 84.44% 84.45% +<.01%
==========================================
Files 27 27
Lines 2090 2091 +1
==========================================
+ Hits 1765 1766 +1
Misses 212 212
Partials 113 113
Continue to review full report at Codecov.
|
@asahasrabuddhe Do you mind adding a test around it? |
Done 👍 |
Thanks for this @vishr Let me know when we can speak about taking the project forward in the right direction |
Absolutely. Let me reach out key contributors and figure it out.
… On Jan 1, 2020, at 9:33 AM, Ajitem Sahasrabuddhe ***@***.***> wrote:
Thanks for this @vishr <https://github.com/vishr> Let me know when we can speak about taking the project forward in the right direction
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#1467?email_source=notifications&email_token=AACMVNFY7NYO3LJSBVTPADTQ3THXLA5CNFSM4KBTR6RKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH5JHCY#issuecomment-570069899>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACMVNA6ZENIAEE23RJPD3TQ3THXLANCNFSM4KBTR6RA>.
|
Fixes #1466
@vishr I would appreciate you taking a look at this and merging this. This is a major regression and I think people would appreciate a quick release fixing this...