Skip to content

[ISSUE 3932] refactor MotanServiceEventListener#3957

Closed
misaya295 wants to merge 1 commit intoapache:masterfrom
misaya295:issuse-#3932
Closed

[ISSUE 3932] refactor MotanServiceEventListener#3957
misaya295 wants to merge 1 commit intoapache:masterfrom
misaya295:issuse-#3932

Conversation

@misaya295
Copy link
Copy Markdown
Contributor

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
Copy Markdown

codecov-commenter commented Sep 17, 2022

Codecov Report

Merging #3957 (56e2479) into master (14ec556) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3957      +/-   ##
============================================
- Coverage     70.16%   70.13%   -0.03%     
+ Complexity     6678     6674       -4     
============================================
  Files           923      923              
  Lines         25071    25071              
  Branches       2264     2264              
============================================
- Hits          17590    17583       -7     
- Misses         6112     6117       +5     
- Partials       1369     1371       +2     
Impacted Files Coverage Δ
...controller/ShenyuClientHttpRegistryController.java 77.77% <0.00%> (-22.23%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 52.94% <0.00%> (-11.77%) ⬇️
...henyu/admin/service/impl/UpstreamCheckService.java 62.66% <0.00%> (-1.34%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 62.76% <0.00%> (-1.07%) ⬇️

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

@loongs-zhang
Copy link
Copy Markdown
Member

please check CI.

@misaya295 misaya295 closed this Sep 18, 2022
@misaya295 misaya295 deleted the issuse-#3932 branch September 18, 2022 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants