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

Drop Ruby 2.5 support #123

Merged
merged 1 commit into from
Aug 2, 2023
Merged

Conversation

archanaserver
Copy link
Contributor

For refernce: #122 (comment)

@ehelms
Copy link
Member

ehelms commented Jul 12, 2023

@archanaserver Can you rebase this?

.github/workflows/ruby.yml Outdated Show resolved Hide resolved
@ehelms
Copy link
Member

ehelms commented Jul 22, 2023

I am proposing we drop the smart_proxy_dynflow Jenkins tests -- theforeman/jenkins-jobs#346

@archanaserver
Copy link
Contributor Author

I am proposing we drop the smart_proxy_dynflow Jenkins tests -- theforeman/jenkins-jobs#346

In a favour of this one, I have raised a new PR to drop the Jenkins changes #124.

@ehelms
Copy link
Member

ehelms commented Jul 27, 2023

Interesting, something triggered an update to a version of minitest where all the MiniTest should now be Minitest -- could you look at doing that in a separate PR? I don't think it has to do with this change in particular.

@archanaserver
Copy link
Contributor Author

Interesting, something triggered an update to a version of minitest where all the MiniTest should now be Minitest -- could you look at doing that in a separate PR? I don't think it has to do with this change in particular.

Raised the changes here #125

@ehelms
Copy link
Member

ehelms commented Jul 31, 2023

@archanaserver This can be rebased now

@ekohl ekohl changed the title Upgrade Ruby version Drop Ruby 2.5 support Aug 1, 2023
@ekohl
Copy link
Member

ekohl commented Aug 1, 2023

@adamruzicka can you help understand why this one test fails?

@adamruzicka
Copy link
Contributor

Maybe if this was reproducible. Last three runs in #126 with the same changes as here were successful and I haven't succeeded in reproducing it locally. When I rerun the tests here it passed.

@ekohl
Copy link
Member

ekohl commented Aug 1, 2023

IIRC minitest also has test order randomization, but perhaps I'm confusing it with rspec. Have you tried rerunning with the same seed to get the same order?

@adamruzicka
Copy link
Contributor

It didn't seem to make any difference

@ehelms
Copy link
Member

ehelms commented Aug 1, 2023

I also could never reproduce it and I only ever saw it fail on Jenkins (which is no longer an issue?).

@ekohl
Copy link
Member

ekohl commented Aug 1, 2023

This failure was in GitHub Actions.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

@adamruzicka shall we merge this?

@adamruzicka adamruzicka merged commit 46cdbc2 into theforeman:master Aug 2, 2023
4 of 5 checks passed
@adamruzicka
Copy link
Contributor

Thank you everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants