Skip to content

[type:fix] fix e2e error. #5152

Merged
moremind merged 19 commits intoapache:masterfrom
yunlongn:fix_e2e_error_0911
Sep 14, 2023
Merged

[type:fix] fix e2e error. #5152
moremind merged 19 commits intoapache:masterfrom
yunlongn:fix_e2e_error_0911

Conversation

@yunlongn
Copy link
Copy Markdown
Member

[type:fix] fix e2e error.

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 13, 2023

Codecov Report

Merging #5152 (658d9b1) into master (e3f3b06) will increase coverage by 0.04%.
The diff coverage is 100.00%.

❗ Current head 658d9b1 differs from pull request most recent head 8be8532. Consider uploading reports for the commit 8be8532 to get more accurate results

@@             Coverage Diff              @@
##             master    #5152      +/-   ##
============================================
+ Coverage     61.06%   61.10%   +0.04%     
- Complexity     8470     8472       +2     
============================================
  Files          1242     1242              
  Lines         37347    37347              
  Branches       3523     3523              
============================================
+ Hits          22806    22821      +15     
+ Misses        12579    12563      -16     
- Partials       1962     1963       +1     
Files Changed Coverage Δ
.../plugin/base/cache/CommonPluginDataSubscriber.java 66.03% <100.00%> (ø)
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 58.44% <100.00%> (+12.98%) ⬆️

... and 2 files with indirect coverage changes

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

@moremind moremind merged commit be65a99 into apache:master Sep 14, 2023
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.

4 participants