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
optimize: fix jackson dependencies in TransactionContextFilterTest #4544
Conversation
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.
请将pr登记到2.0.0.md中
This pull request introduces 1 alert and fixes 6 when merging e1eb9c4 into 4c88984 - view on LGTM.com new alerts:
fixed alerts:
|
Codecov Report
@@ Coverage Diff @@
## develop #4544 +/- ##
==========================================
Coverage 49.35% 49.35%
Complexity 4073 4073
==========================================
Files 731 731
Lines 25465 25465
Branches 3156 3156
==========================================
Hits 12569 12569
+ Misses 11559 11558 -1
- Partials 1337 1338 +1
|
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
@robynron Pls resolve the conflicts. |
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
Ⅰ. Describe what this PR did
优化了测试用例TransactionContextFilterTest中jackson包依赖
Ⅱ. Does this pull request fix one issue?
fixes #4540
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews