Skip to content

Remove Ruby version constraint workarounds #2386

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

Merged
merged 2 commits into from
Oct 10, 2020

Conversation

pirj
Copy link
Member

@pirj pirj commented Sep 24, 2020

The point of this PR is to remove workaround previously needed to work around Rubygems API not providing necessary gem Ruby constraints under certain conditions.

The problem was presumably fixed in Rubygems.
A couple of passed builds have no odd gem version resolutions resulting in Ruby version incompatibility.

Follow-up to #2380

Related to rubygems/rubygems#3463

@pirj pirj self-assigned this Sep 24, 2020
@pirj pirj marked this pull request as draft September 24, 2020 22:06
@pirj pirj force-pushed the remove-ruby-version-constraint-workarounds branch from b8253c4 to ed6a963 Compare October 10, 2020 16:39
@pirj pirj marked this pull request as ready for review October 10, 2020 16:40
@pirj pirj requested review from JonRowe and benoittgt October 10, 2020 16:40
@pirj
Copy link
Member Author

pirj commented Oct 10, 2020

Green!
image

We used to specify them to pin them.
@pirj
Copy link
Member Author

pirj commented Oct 10, 2020

It was a nice excursion back into time when we had 72 build jobs.
https://travis-ci.org/github/rspec/rspec-rails/builds/734613066 is green
Squashed commits and waiting for yet another green build.

@pirj pirj force-pushed the remove-ruby-version-constraint-workarounds branch from e3c020f to 29aeb00 Compare October 10, 2020 20:01
@pirj pirj requested a review from JonRowe October 10, 2020 20:02
Copy link
Member

@JonRowe JonRowe left a comment

Choose a reason for hiding this comment

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

If it goes green feel free to merge

@pirj pirj merged commit 487fa59 into 3-9-maintenance Oct 10, 2020
@pirj pirj deleted the remove-ruby-version-constraint-workarounds branch October 10, 2020 20:59
@benoittgt
Copy link
Member

Yes!! Thanks

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.

3 participants