-
Notifications
You must be signed in to change notification settings - Fork 4.8k
[Bug] fix switch condition #15228
New issue
Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “No 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? No Sign in to your account
[Bug] fix switch condition #15228
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #15228 +/- ##
============================================
+ Coverage 37.99% 38.02% +0.03%
- Complexity 4640 4645 +5
============================================
Files 1279 1279
Lines 44490 44500 +10
Branches 4794 4797 +3
============================================
+ Hits 16902 16923 +21
+ Misses 25733 25717 -16
- Partials 1855 1860 +5 ☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! |
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.
👍
Co-authored-by: Eric Gao <ericgao.apache@gmail.com>
Purpose of the pull request
add switch args check to avoid illegal condition.
Brief change log
SwitchLogicTask
Verify this pull request
SwitchTaskUtilsTest