-
Notifications
You must be signed in to change notification settings - Fork 625
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
[ISSUE #2167]fix_encoding #2168
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.
Welcome to the Apache EventMesh (incubating) community!!
This is your first PR in our project. We're very excited to have you onboard contributing. Your contributions are greatly appreciated!
Please make sure that the changes are covered by tests.
We will be here shortly.
Let us know if you need any help!
Want to get closer to the community?
Mailing Lists:
Name | Description | Subscribe | Unsubscribe | Archive |
---|---|---|---|---|
Users | User support and questions mailing list | Subscribe | Unsubscribe | Mail Archives |
Development | Development related discussions | Subscribe | Unsubscribe | Mail Archives |
Commits | All commits to repositories | Subscribe | Unsubscribe | Mail Archives |
...tor-redis/src/test/java/org/apache/eventmesh/connector/redis/consumer/RedisConsumerTest.java
Show resolved
Hide resolved
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.
please check the code
Codecov Report
@@ Coverage Diff @@
## master #2168 +/- ##
=========================================
Coverage 11.79% 11.79%
Complexity 899 899
=========================================
Files 468 468
Lines 27863 27862 -1
Branches 3010 3010
=========================================
Hits 3287 3287
+ Misses 24279 24278 -1
Partials 297 297
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Fixes #2167 .
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Reliance on default encoding
Modifications
add encode
Documentation