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

Fix ruby-head CI #44152

Merged
merged 1 commit into from Jan 12, 2022
Merged

Fix ruby-head CI #44152

merged 1 commit into from Jan 12, 2022

Conversation

casperisfine
Copy link
Contributor

@casperisfine casperisfine commented Jan 12, 2022

rbtree no longer compile cleanly on Ruby 3.2, @zzak fixed it at https://github.com/zzak/rbtree and I added a gemspec so it can be used as a git gem, let's see if it works.

blade uses File.exist? that was removed in 3.2: javan/blade#12

@casperisfine casperisfine changed the title Use a rbtree fork to fix ruby-head CI Fix ruby-head CI Jan 12, 2022
@casperisfine casperisfine marked this pull request as ready for review January 12, 2022 10:02
They both need very minor fixes.
@casperisfine
Copy link
Contributor Author

@zzak @matthewd we should be good to go. Up to you wether we should merge now, or wait for upstream to cut releases, I don't mind either way.

@casperisfine
Copy link
Contributor Author

Got the 👍 on basecamp. Merging.

@byroot byroot merged commit e2865d5 into rails:main Jan 12, 2022
yahonda added a commit to yahonda/rails that referenced this pull request Feb 7, 2022
https://github.com/mame/rbtree is mirror of rbtree gem
and rbtree 0.4.5 has been released to support Ruby 3.2.0dev.
https://rubygems.org/gems/rbtree/versions/0.4.5

Related to rails#44152
public-rant pushed a commit to opensource-rant/rails that referenced this pull request Feb 17, 2022
https://github.com/mame/rbtree is mirror of rbtree gem
and rbtree 0.4.5 has been released to support Ruby 3.2.0dev.
https://rubygems.org/gems/rbtree/versions/0.4.5

Related to rails#44152
yahonda added a commit to yahonda/rails that referenced this pull request Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants