Skip to content

Conversation

@misaya295
Copy link
Contributor

@misaya295 misaya295 commented Sep 6, 2022

About #3921

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 Sep 6, 2022

Codecov Report

Merging #3922 (49c7265) into master (6fee634) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3922      +/-   ##
============================================
- Coverage     70.44%   70.42%   -0.02%     
+ Complexity     6767     6766       -1     
============================================
  Files           923      923              
  Lines         25373    25373              
  Branches       2310     2310              
============================================
- Hits          17873    17869       -4     
- Misses         6129     6133       +4     
  Partials       1371     1371              
Impacted Files Coverage Δ
...controller/ShenyuClientHttpRegistryController.java 77.77% <0.00%> (-22.23%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 52.94% <0.00%> (-11.77%) ⬇️

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

@yu199195 yu199195 merged commit 9f39f55 into apache:master Sep 6, 2022
@yu199195 yu199195 added this to the 2.5.1 milestone Sep 6, 2022
@yu199195 yu199195 added the admin label Sep 6, 2022
@misaya295 misaya295 deleted the issues-#3921 branch September 9, 2022 09:46
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