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

Gemfile supports ruby 3.3 #23204

Merged
merged 1 commit into from
Sep 27, 2024
Merged

Gemfile supports ruby 3.3 #23204

merged 1 commit into from
Sep 27, 2024

Conversation

kbrock
Copy link
Member

@kbrock kbrock commented Sep 24, 2024

This allows tests to run with ruby3.3

Other non core repos were complaining when bringing in this Gemfile.

@kbrock
Copy link
Member Author

kbrock commented Sep 24, 2024

failure is the wrong version of get_fasttexxt

fixed with: #23205

@miq-bot
Copy link
Member

miq-bot commented Sep 25, 2024

This pull request is not mergeable. Please rebase and repush.

@kbrock kbrock changed the title Test with ruby Gemfile supports ruby 3.3 Sep 26, 2024
@kbrock
Copy link
Member Author

kbrock commented Sep 26, 2024

update:

  • rspec were already merged
  • Just upgrading Gemfile and ci

@kbrock kbrock force-pushed the test_with_ruby branch 2 times, most recently from 21f47bf to ad7d36a Compare September 26, 2024 23:13
@kbrock
Copy link
Member Author

kbrock commented Sep 27, 2024

@miq-bot cross-repo-tests /all

miq-bot pushed a commit to ManageIQ/manageiq-cross_repo-tests that referenced this pull request Sep 27, 2024
Gemfile Show resolved Hide resolved
@Fryguy Fryguy self-assigned this Sep 27, 2024
testing with ruby 3.3
These changes allow other repos to run with ruby 3.3
@miq-bot
Copy link
Member

miq-bot commented Sep 27, 2024

Checked commit kbrock@0bb23f1 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@kbrock
Copy link
Member Author

kbrock commented Sep 27, 2024

update:

  • changed from Gemfile#ruby back to RUBY_VERSION for ruby checking. brakeman was raising errors when running ruby 3.0.

update:

  • added a warning to Gemfile if ruby is 3.2 or 3.3

Thinking kicking off cross-repo tests is not necessary with this change

Copy link
Member

@agrare agrare left a comment

Choose a reason for hiding this comment

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

Ran a brief sanity check locally with ruby3.3 and so far everything looks good 👍

Copy link
Member

@jrafanie jrafanie left a comment

Choose a reason for hiding this comment

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

LGTM :shipit: Let's see how rough testing 4 rubies will be on core.

We can always be selective if it becomes a problem.

@jrafanie jrafanie merged commit b6ed619 into ManageIQ:master Sep 27, 2024
12 checks passed
@jrafanie jrafanie assigned jrafanie and unassigned Fryguy Sep 27, 2024
@kbrock kbrock deleted the test_with_ruby branch September 27, 2024 20:45
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.

5 participants