fix: hotfix for zip2 2.6.0 breaking changes

[AyushAgrawal-A2]

Description

fixes failing CI builds due to breaking changes in zip2 2.6.0

ref:

• Builds failing due to semver violation in Zip tafia/calamine#492
• Dont start zip-rs/zip2#332 (comment)

[qa-wolf]

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

[github-actions]

Preview - Build & Deploy Images

✅ Build images
✅ Deploy images

🕒 Last deployed: Apr 03, 2025 at 09:45 PM UTC

🔗 URL: https://ayush-fix-ci-zip2.quadratic-preview.com

[graphite-app]

There appears to be a duplicate forward slash in the GitHub URL. The URL should be https://github.com/zip-rs/zip2 instead of https://github.com//zip-rs/zip2. This typo appears in multiple patch sections throughout the PR.

Suggested change

	zip = { git = "https://github.com//zip-rs/zip2", rev = "77cde6a" }
	zip = { git = "https://github.com/zip-rs/zip2", rev = "77cde6a" }

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

[ddimaria]

@graphite-app see https://gatowololo.github.io/blog/cargo-patch/#crab-sharp-edge-git-dependency-with-extra

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (77e33da) to head (f1f3c7b).
Report is 8 commits behind head on qa.

Additional details and impacted files

@@            Coverage Diff             @@
##               qa   #2652       +/-   ##
==========================================
- Coverage   90.41%       0   -90.42%     
==========================================
  Files         383       0      -383     
  Lines       86288       0    -86288     
==========================================
- Hits        78014       0    -78014     
+ Misses       8274       0     -8274     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[graphite-app]

There's an inconsistency in the patch URL configuration. In this file, the patch URL is specified as [patch.'https://github.com/zip-rs/zip2'], while in other files it's configured as [patch.'https://github.com/zip-rs/zip'].

This inconsistency could lead to dependency resolution issues. Consider updating this to match the pattern used in the other files:

[patch.'https://github.com/zip-rs/zip']
zip = { git = "https://github.com/zip-rs/zip2", rev = "77cde6a" }

Also note the double slash in the git URL (https://github.com//zip-rs/zip2), which should be corrected to a single slash.

Suggested change

	[patch.'https://github.com/zip-rs/zip2']
	zip = { git = "https://github.com//zip-rs/zip2", rev = "77cde6a" }
	[patch.'https://github.com/zip-rs/zip']
	zip = { git = "https://github.com/zip-rs/zip2", rev = "77cde6a" }

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.