-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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(controller): support float for param value #4490
Conversation
413d81b
to
4544657
Compare
5861cc1
to
4c4c336
Compare
You have a failing unit test? |
4c4c336
to
3cb9f5a
Compare
@alexec probably flake. Fixed now |
I was just about the send a similar PR to fix this as well from a working branch and then found this one. Thanks @arghya88! |
@terrytangyuan good comment - thank you! |
9de7f91
to
0071fc5
Compare
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.
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.
We should back-port to v2.12 if easy. v3 if not.
b65f396
to
1b93935
Compare
c3d9fc9
to
458b646
Compare
Signed-off-by: Arghya Sadhu <[email protected]>
458b646
to
dd30b4f
Compare
Needs backporting to v2.12 |
@alexec I will submit a PR for that |
No need. Will do this for you. |
Signed-off-by: Arghya Sadhu <[email protected]> Signed-off-by: Alex Capras <[email protected]>
Checklist:
Fixes #4478