Skip to content

Conversation

@piksel
Copy link
Member

@piksel piksel commented Oct 9, 2021

Follow up to #574 that implements DisposeAsync and "true" async write support to GzipOutputStream.

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

@codecov
Copy link

codecov bot commented Oct 9, 2021

Codecov Report

Merging #672 (e61251a) into master (519ed73) will increase coverage by 0.10%.
The diff coverage is 90.32%.

@@            Coverage Diff             @@
##           master     #672      +/-   ##
==========================================
+ Coverage   74.60%   74.70%   +0.10%     
==========================================
  Files          72       72              
  Lines        8457     8465       +8     
==========================================
+ Hits         6309     6324      +15     
+ Misses       2148     2141       -7     
Impacted Files Coverage Δ
...ib/Zip/Compression/Streams/DeflaterOutputStream.cs 82.47% <ø> (+7.70%) ⬆️
...c/ICSharpCode.SharpZipLib/GZip/GzipOutputStream.cs 86.48% <90.32%> (-1.02%) ⬇️

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

@piksel piksel force-pushed the feat/async-gzipos branch from 05557d3 to e63c41f Compare October 9, 2021 19:09
@piksel piksel added this to the v1.4 milestone Oct 10, 2021
For ZipStreamAsyncTest we are explicitly testing for `await using`, so
it's now being marked as ignored and a dummy await is added to skip warnings

For GzipAsyncTests we are testing multiple async methods, so variants are
added that simply skips the `await using`/`DisposeAsync` and calls the
underlying `FinishAsync`
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.

2 participants