Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

optimize: h2 dependency adds test scope #6906

Merged
merged 5 commits into from
Oct 10, 2024
Merged

Conversation

yougecn
Copy link
Contributor

@yougecn yougecn commented Oct 9, 2024

h2 dependency adds test scope

@funky-eyes funky-eyes added this to the 2.3.0 milestone Oct 9, 2024
@funky-eyes funky-eyes changed the title optimize:h2 dependency adds test scope optimize: h2 dependency adds test scope Oct 9, 2024
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.33%. Comparing base (679f86e) to head (cdf6dea).
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##                2.x    #6906   +/-   ##
=========================================
  Coverage     52.32%   52.33%           
  Complexity     6523     6523           
=========================================
  Files          1106     1106           
  Lines         38716    38716           
  Branches       4588     4588           
=========================================
+ Hits          20259    20261    +2     
+ Misses        16465    16464    -1     
+ Partials       1992     1991    -1     

see 1 file with indirect coverage changes

Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes merged commit 3b8ccfc into apache:2.x Oct 10, 2024
7 checks passed
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.

2 participants