Skip to content

Add support for rubyzip 3.x#629

Merged
simonoff merged 2 commits into
roo-rb:masterfrom
jukra:rubyzip3
Jul 30, 2025
Merged

Add support for rubyzip 3.x#629
simonoff merged 2 commits into
roo-rb:masterfrom
jukra:rubyzip3

Conversation

@jukra
Copy link
Copy Markdown
Contributor

@jukra jukra commented Jul 30, 2025

Summary

Added support for rubyzip 3.0.0 and made it the minimum required version.

Other Information

Related issue: #621
Rubyzip upgrade instructions: https://github.com/rubyzip/rubyzip/wiki/Updating-to-version-3.x#open

@simonoff
Copy link
Copy Markdown
Member

@jukra thank you for yyour help. Can you please add a changelog?

@jukra
Copy link
Copy Markdown
Contributor Author

jukra commented Jul 30, 2025

@jukra thank you for yyour help. Can you please add a changelog?

@simonoff Done!

@jukra jukra changed the title Add support for rubyzip 3 Add support for rubyzip 3.x Jul 30, 2025
@jukra
Copy link
Copy Markdown
Contributor Author

jukra commented Jul 30, 2025

Also added the '< 4.0.0' for gemspec

@simonoff
Copy link
Copy Markdown
Member

Also added the '< 4.0.0' for gemspec

Thank you.

But i think we need to notice that old rubyzip will not work anymore. If you want to add just a support for 3.0 then i think better to have a detection of version and different behaviour depend on it. Or fully deprecation of < 3.0 versions.

@jukra
Copy link
Copy Markdown
Contributor Author

jukra commented Jul 30, 2025

Thank you.

But i think we need to notice that old rubyzip will not work anymore. If you want to add just a support for 3.0 then i think better to have a detection of version and different behaviour depend on it. Or fully deprecation of < 3.0 versions.

Yeah sorry I was unclear, the current requirement is:

spec.add_dependency 'rubyzip', '>= 3.0.0', '< 4.0.0'

so it's fully deprecating < 3.0 versions. I just added the '< 4.0.0' for future.

@simonoff
Copy link
Copy Markdown
Member

Thank you.
But i think we need to notice that old rubyzip will not work anymore. If you want to add just a support for 3.0 then i think better to have a detection of version and different behaviour depend on it. Or fully deprecation of < 3.0 versions.

Yeah sorry I was unclear, the current requirement is:

spec.add_dependency 'rubyzip', '>= 3.0.0', '< 4.0.0'

so it's fully deprecating < 3.0 versions. I just added the '< 4.0.0' for future.

Thank you!

Ok, then also need to mentioned that we dropped support for rubyzip < 3.0

@jukra
Copy link
Copy Markdown
Contributor Author

jukra commented Jul 30, 2025

@simonoff added now a note about that as well to changelog d9e83d6

@simonoff simonoff merged commit 80146fa into roo-rb:master Jul 30, 2025
7 checks passed
@simonoff
Copy link
Copy Markdown
Member

@jukra thank you!

@ecentell-CPF
Copy link
Copy Markdown

Do you know when this will be published against rubygems.org?

@ecentell-CPF
Copy link
Copy Markdown

Just checking in on publishing this version? If this is an inappropriate place to ask about rubygems.org publish please let me know.

@jukra
Copy link
Copy Markdown
Contributor Author

jukra commented Oct 1, 2025

I think @simonoff is releasing soon #630

@simonoff
Copy link
Copy Markdown
Member

simonoff commented Oct 1, 2025

Released!

@PikachuEXE
Copy link
Copy Markdown

Please also check roo-rb/roo-xls#64 for roo-xls

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.

4 participants