-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
expression: fix a bug that the coercibility of user variable is wrong #17676
Conversation
Signed-off-by: wjhuang2016 <[email protected]>
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
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
Sorry @bb7133, you don't have permission to trigger auto merge event on this branch. You are not a committer for this part |
/merge |
Sorry @wjhuang2016, you don't have permission to trigger auto merge event on this branch. You are not a committer for this part |
/merge |
Sorry @wjhuang2016, you don't have permission to trigger auto merge event on this branch. You are not a committer for this part |
Codecov Report
@@ Coverage Diff @@
## master #17676 +/- ##
===========================================
Coverage 79.4949% 79.4949%
===========================================
Files 524 524
Lines 141887 141887
===========================================
Hits 112793 112793
Misses 19994 19994
Partials 9100 9100 |
/merge |
Sorry @wjhuang2016, you don't have permission to trigger auto merge event on this branch. You are not a committer for this part |
/merge |
/run-all-tests |
@wjhuang2016 merge failed. |
Signed-off-by: sre-bot <[email protected]>
cherry pick to release-4.0 in PR #17890 |
Signed-off-by: wjhuang2016 [email protected]
What problem does this PR solve?
Issue Number: close #17675
Problem Summary:
We didn't handle the coercibility for user variables.
What is changed and how it works?
Set the coercibility of user variables to CoercibilityImplicit.
Related changes
Check List
Tests
Side effects
Release note