Skip to content
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

Merged
merged 4 commits into from Dec 14, 2023

Conversation

lacostej
Copy link
Collaborator

@lacostej lacostej commented Dec 1, 2023

3 tests are failing in ruby 3.3.0-preview3

./fastlane/spec/plugins_specs/plugin_generator_spec.rb:113 
./fastlane_core/spec/configuration_file_spec.rb:115 
./frameit/spec/strings_parser_spec.rb:49

ruby 3.3. isn't yet available on circleci though.

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
  • I've read the Contribution Guidelines
  • I've added or updated relevant unit tests.

Motivation and Context

Description

Testing Steps

rspec

@lacostej lacostej changed the title Fix/ruby 3 3 compatibility Support ruby 3.3 Dec 11, 2023
Copy link
Member

@rogerluan rogerluan left a 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?

fastlane/spec/plugins_specs/plugin_generator_spec.rb Outdated Show resolved Hide resolved
@rogerluan rogerluan changed the title Support ruby 3.3 [fastlane_core] add support to Ruby 3.3 Dec 13, 2023
Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
@lacostej
Copy link
Collaborator Author

Is your plan to get this merged even before CircleCI supports Ruby 3.3 envs?

I would. Then we are ready to add ruby 3.3 as soon as it is available on circle.

Copy link
Member

@rogerluan rogerluan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM @lacostej 🚀

@lacostej lacostej merged commit 4eb636f into fastlane:master Dec 14, 2023
2 checks passed
@lacostej lacostej deleted the fix/ruby_3_3_compatibility branch December 14, 2023 13:04
SubhrajyotiSen pushed a commit to KeepTruckin/fastlane that referenced this pull request Jan 17, 2024
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants