search

ladjs / superagent

Ajax for Node.js and browsers (JS HTTP client). Maintained for @forwardemail, @ladjs, @spamscanner, @breejs, @cabinjs, and @lassjs.
https://ladjs.github.io/superagent/
MIT License
16.56k stars 1.33k forks source link

ci: fixed broken ci #1744

Closed lamweili closed 1 year ago

lamweili commented 1 year ago

fixed ci

fixed test causing ci to fail

security fixes

codecov-commenter commented 1 year ago

Codecov Report

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

:exclamation: 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 commented 1 year ago

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