-
Notifications
You must be signed in to change notification settings - Fork 542
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: ! (reverse) operator not handled correctly #2364
Conversation
Great! Hi, @SkyeYoung can you add the test cases, please refer to: |
I'll try to do it. |
@guoqqqi I added test cases, pls have a look. : ) |
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
Codecov Report
@@ Coverage Diff @@
## master #2364 +/- ##
==========================================
- Coverage 69.91% 68.50% -1.41%
==========================================
Files 184 131 -53
Lines 7279 3420 -3859
Branches 830 828 -2
==========================================
- Hits 5089 2343 -2746
+ Misses 1896 1077 -819
+ Partials 294 0 -294
Flags with carried forward coverage won't be shown. Click here to find out more.
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.
LGTM!
* origin: feat: release 2.11.0 (apache#2361) feat: add data loader framework (apache#2371) feat: improve Consumer module (apache#2327) fix: ! (reverse) operator not handled correctly (apache#2364) chore: modify the select upstream field to upstream (apache#2344) feat: support protobuf on Web (apache#2320) chore: Extract paging related functions into standalone hook (apache#2334) feat: basic support Apache APISIX 2.12.1 (apache#2315) # Conflicts: # web/src/helpers.tsx # web/yarn.lock
Please answer these questions before submitting a pull request, or your PR will get closed.
Why submit this pull request?
What changes will this PR take into?
Correct handling ! (reverse) operator
Related issues
fix #2166
Checklist: