Skip to content

Conversation

@xcsnx
Copy link
Member

@xcsnx xcsnx commented Nov 29, 2022

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 Nov 29, 2022

Codecov Report

Merging #4223 (8c7595f) into master (35c30d0) will increase coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4223      +/-   ##
============================================
+ Coverage     70.56%   70.66%   +0.09%     
- Complexity     7254     7267      +13     
============================================
  Files           971      971              
  Lines         27083    27083              
  Branches       2408     2408              
============================================
+ Hits          19110    19137      +27     
+ Misses         6512     6482      -30     
- Partials       1461     1464       +3     
Impacted Files Coverage Δ
...java/org/apache/shenyu/common/utils/PathUtils.java 50.00% <0.00%> (ø)
...java/org/apache/shenyu/common/utils/GsonUtils.java 66.06% <0.00%> (+1.21%) ⬆️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 62.76% <0.00%> (+8.51%) ⬆️
...java/org/apache/shenyu/common/utils/JsonUtils.java 67.30% <0.00%> (+11.53%) ⬆️
...ruptor/RegisterClientServerDisruptorPublisher.java 64.70% <0.00%> (+11.76%) ⬆️
...java/org/apache/shenyu/common/utils/SignUtils.java 100.00% <0.00%> (+15.78%) ⬆️
...controller/ShenyuClientHttpRegistryController.java 100.00% <0.00%> (+22.22%) ⬆️
.../java/org/apache/shenyu/common/utils/UriUtils.java 78.57% <0.00%> (+28.57%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@li-keguo li-keguo merged commit 6a7f0ad into apache:master Nov 29, 2022
@xcsnx xcsnx deleted the csn-dev-brach branch November 29, 2022 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants