Coverage decreased (-0.04%) to 91.339% when pulling 210a26a on aeroastro:feature/tempfile-directory into 98f4f2e on rubyzip:master.

Coverage decreased (-0.04%) to 91.339% when pulling 258ef02 on aeroastro:feature/tempfile-directory into 98f4f2e on rubyzip:master.

Should I maintain or increase coverage for this patch to be merged?

Since this patch implements correct (or recommended) arguments to Ruby built-in library, IMHO there seems to be no strong requirements to keep the coverage.

Also, the fact the coverage decrease from https://coveralls.io/builds/10052092 (recent master push) to https://coveralls.io/builds/12143274 (this branch) is due to increased number of relevant lines from 2031 to 2032. This is actually triggered by counting end line as relevant lines here https://coveralls.io/builds/12143274/source?filename=lib%2Fzip%2Fstreamable_stream.rb#L9. It seems like there are some issues on coverage calculation side.

I just would like to stop temporary files from filling up application's root directory. 🐱

@simonoff

Hello.

It's been a while since I submitted this PR, and I'm wondering if there is anything I could do on my end. I totally understand you are busy and require more time to consider, it is much appreciated if you could give me a response.

Thank you very much!!!