Skip to content

Conversation

@moremind
Copy link
Member

@moremind moremind commented Jul 31, 2022

about #3773 task2 and task3,add test for #3158 and #3687

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link

codecov-commenter commented Jul 31, 2022

Codecov Report

Merging #3779 (59fe0a2) into master (04142f4) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3779      +/-   ##
============================================
+ Coverage     64.23%   64.25%   +0.02%     
- Complexity     6039     6045       +6     
============================================
  Files           900      900              
  Lines         24673    24673              
  Branches       2237     2237              
============================================
+ Hits          15849    15854       +5     
+ Misses         7425     7421       -4     
+ Partials       1399     1398       -1     
Impacted Files Coverage Δ
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 63.82% <0.00%> (+1.06%) ⬆️
...ruptor/RegisterClientServerDisruptorPublisher.java 64.70% <0.00%> (+11.76%) ⬆️
...controller/ShenyuClientHttpRegistryController.java 100.00% <0.00%> (+22.22%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us.

@loongs-zhang loongs-zhang merged commit c7cc683 into apache:master Jul 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants