Skip to content

[rust] update zip to v8#17485

Merged
titusfortner merged 1 commit into
trunkfrom
rust_zip
May 16, 2026
Merged

[rust] update zip to v8#17485
titusfortner merged 1 commit into
trunkfrom
rust_zip

Conversation

@titusfortner
Copy link
Copy Markdown
Member

🔗 Related Issues

Should have been included in #17472 but agent was being conservative because of the major version bumps.

💥 What does this PR do?

Updates zip dependency, shouldn't be any issues with this one, we'll verify with tests

@qodo-code-review
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Update Rust zip dependency to version 8

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Updates zip dependency from v5.1.1 to v8
• Enables compatibility with latest zip library features
• Maintains existing deflate compression feature configuration
Diagram
flowchart LR
  A["zip v5.1.1"] -- "upgrade" --> B["zip v8"]
  B -- "maintains" --> C["deflate feature"]
Loading

Grey Divider

File Changes

1. rust/Cargo.toml Dependencies +1/-1

Upgrade zip crate to version 8

• Updates zip dependency version from 5.1.1 to 8
• Preserves deflate feature flag configuration
• No changes to other dependencies or features

rust/Cargo.toml


Grey Divider

Qodo Logo

@selenium-ci selenium-ci added C-rust Rust code is mostly Selenium Manager B-manager Selenium Manager labels May 16, 2026
@qodo-code-review
Copy link
Copy Markdown
Contributor

qodo-code-review Bot commented May 16, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@titusfortner titusfortner merged commit 378a0f9 into trunk May 16, 2026
40 checks passed
@titusfortner titusfortner deleted the rust_zip branch May 16, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-manager Selenium Manager C-rust Rust code is mostly Selenium Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants