Skip to content

ci: fixed broken ci#1744

Merged
titanism merged 6 commits into
forwardemail:masterfrom
lamweili:fix-ci
Oct 4, 2022
Merged

ci: fixed broken ci#1744
titanism merged 6 commits into
forwardemail:masterfrom
lamweili:fix-ci

Conversation

@lamweili

@lamweili lamweili commented Oct 3, 2022

Copy link
Copy Markdown
Contributor

@lamweili lamweili marked this pull request as draft October 3, 2022 16:22
@codecov-commenter

codecov-commenter commented Oct 3, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1744 (c122c42) into master (29fd1f9) will not change coverage.
The diff coverage is n/a.

❗ Current head c122c42 differs from pull request most recent head 14b2386. Consider uploading reports for the commit 14b2386 to get more accurate results

@@           Coverage Diff           @@
##           master    #1744   +/-   ##
=======================================
  Coverage   94.09%   94.09%           
=======================================
  Files          14       14           
  Lines        1135     1135           
=======================================
  Hits         1068     1068           
  Misses         67       67           
Impacted Files Coverage Δ
src/node/http2wrapper.js 96.26% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@lamweili lamweili marked this pull request as ready for review October 3, 2022 18:26
@lamweili

lamweili commented Oct 3, 2022

Copy link
Copy Markdown
Contributor Author

@titanism, please review this if you have this time. 🙏
No release is needed as this doesn't affect the code.

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