Skip to content

fix: Specify flate2 dependency of the deflate-flate2 feature.#345

Merged
Pr0methean merged 3 commits into
zip-rs:masterfrom
anforowicz:fix-building-with-just-deflate-flate2-feature
May 10, 2025
Merged

fix: Specify flate2 dependency of the deflate-flate2 feature.#345
Pr0methean merged 3 commits into
zip-rs:masterfrom
anforowicz:fix-building-with-just-deflate-flate2-feature

Conversation

@anforowicz

Copy link
Copy Markdown
Contributor

PTAL?

Fixes #344


RE: PR requirements

  • I don't know how to grant @Pr0methean write access to the source branch... I guess I could send an invite to become a collaborator, but hopefully that won't be necessary, since the changes in this PR are quite small + the fork has been synced today
  • The PR targets zip-rs/zip2 and not zip-rs/zip-old
  • I don't plan to become a regular contributor, so I haven't yet enabled workflows at https://github.com/anforowicz/zip2/actions. I hope that it's okay if I upload this PR as a "draft" and check the CI results here.
  • AFAICT the PR title does conform to Conventional Commits and starts with one of the types specified by the Angular convention.

@anforowicz anforowicz marked this pull request as draft May 7, 2025 18:53
@Pr0methean Pr0methean marked this pull request as ready for review May 9, 2025 23:13
@Pr0methean Pr0methean enabled auto-merge May 9, 2025 23:14
@Pr0methean

Copy link
Copy Markdown
Member

Thanks! Since this fixes the build in master that's currently broken, I'll merge it as soon as the fuzz tests and DevSkim pass.

@Pr0methean Pr0methean added this pull request to the merge queue May 10, 2025
Merged via the queue into zip-rs:master with commit 6242a00 May 10, 2025
51 checks passed
@Pr0methean Pr0methean mentioned this pull request May 10, 2025
cosmicexplorer pushed a commit to cosmicexplorer/zip2 that referenced this pull request Sep 30, 2025
…p-rs#345)

* fix: Specify `flate2` dependency of the `deflate-flate2` feature.

Fixes zip-rs#344

* chore: Avoid Clippy's `uninlined_format_args` warning.

This commit helps the CI avoid running into
https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args

Example failure before this commit can be seen at
https://github.com/anforowicz/zip2/actions/runs/14892057603/job/41826536710

---------

Co-authored-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
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.

Build failure when only deflate-flate2 crate feature is enabled

2 participants