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] add support to Ruby 3.3 #21683
[fastlane_core] add support to Ruby 3.3 #21683
Conversation
b74dc53
to
49e0d4a
Compare
…ible with ruby 3.3
…ible with ruby 3.3
…te for internal changes in ruby 3.3 evaluation of __FILE__ within the eval() call
49e0d4a
to
e20f960
Compare
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.
Just some suggestions on wording and inconsistency between __FILE__
and __LINE__
, otherwise LGTM.
Is your plan to get this merged even before CircleCI supports Ruby 3.3 envs?
Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
I would. Then we are ready to add ruby 3.3 as soon as it is available on circle. |
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.
SGTM @lacostej 🚀
* Relax regular expression that captures compilation error to be compatible with ruby 3.3 * Relax regular expression that captures compilation error to be compatible with ruby 3.3 * Change the way we build the test and its expected results to accomodate for internal changes in ruby 3.3 evaluation of __FILE__ within the eval() call * Update fastlane/spec/plugins_specs/plugin_generator_spec.rb Co-authored-by: Roger Oba <rogerluan.oba@gmail.com> --------- Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
3 tests are failing in ruby 3.3.0-preview3
ruby 3.3. isn't yet available on circleci though.
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 validci/circleci
builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)Motivation and Context
Description
Testing Steps
rspec