New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[fastlane-core] Require rubyzip >= 2.0.0 #16660
Conversation
Just saw #15970 (comment). @joshdholtz, do you want me to make it |
Yes please! |
2.x fixes some bugs and also supports only Ruby 2.4 and above, like fastlane. `Zip.validate_entry_sizes = true` was removed because it's the default in rubyzip 2.x. https://github.com/rubyzip/rubyzip/blob/v2.3.0/rubyzip.gemspec#L23 https://github.com/rubyzip/rubyzip/blob/v2.3.0/Changelog.md https://github.com/rubyzip/rubyzip/blob/v2.3.0/Changelog.md#130-2019-09-25 https://github.com/fastlane/fastlane/blob/2.150.0.rc1/fastlane_core/lib/fastlane_core/ipa_file_analyser.rb#L40
6c06b90
to
4c942bb
Compare
Done. |
Thanks. |
Hey @aried3r 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment here 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Congratulations! 🎉 This was released as part of fastlane 2.150.0 🚀
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validMotivation and Context
2.x fixes some bugs and also supports only Ruby 2.4 and above, like fastlane.
Zip.validate_entry_sizes = true
was removed because it's the default in rubyzip 2.x.https://github.com/rubyzip/rubyzip/blob/v2.3.0/rubyzip.gemspec#L23
https://github.com/rubyzip/rubyzip/blob/v2.3.0/Changelog.md
https://github.com/rubyzip/rubyzip/blob/v2.3.0/Changelog.md#130-2019-09-25
https://github.com/fastlane/fastlane/blob/2.150.0.rc1/fastlane_core/lib/fastlane_core/ipa_file_analyser.rb#L40
Edit:
Related PRs:
#16072
#15970
Description
The only direct usage of
rubyzip
I could find was inipa_file_analyzer.rb
, I did not find any usage that should break with this change.Testing Steps
Ran all specs and especially the
ipd_file_analyzer_specs.rb
, which pass withrubyzip
2.3.