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

Lock Rack version due to Grape incompatibility #918

Merged
merged 1 commit into from
Jan 10, 2020

Conversation

marcotc
Copy link
Member

@marcotc marcotc commented Jan 10, 2020

The newly released version of Rack, 2.1.0, is incompatible with the latest version of Grape: ruby-grape/grape#1966

This PR locks Rack to < 2.1.0 until Grape updates the issue with further instructions.

This unblocks our CI builds.

@marcotc marcotc added integrations Involves tracing integrations dev/testing Involves testing processes (e.g. RSpec) labels Jan 10, 2020
@marcotc marcotc requested a review from a team January 10, 2020 20:11
@marcotc marcotc self-assigned this Jan 10, 2020
@dblock
Copy link

dblock commented Jan 10, 2020

I suggest working on a fix in Grape rather than hiding this. Maybe try a PR for the fix there?

Copy link
Contributor

@delner delner left a comment

Choose a reason for hiding this comment

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

Thanks for the quick pin @marcotc . 👍

@delner
Copy link
Contributor

delner commented Jan 10, 2020

@dblock We will try to fix the bug at the source if we can; this change is just to unblock our CI pipeline, which tests Grape and our instrumentation of it.

@marcotc marcotc merged commit 1fefa46 into master Jan 10, 2020
@marcotc marcotc deleted the test/grape-rack-incompatibility branch January 10, 2020 20:20
@marcotc marcotc mentioned this pull request Jan 15, 2020
@marcotc marcotc added this to the 0.31.1 milestone Jan 16, 2020
marcotc added a commit that referenced this pull request Jan 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/testing Involves testing processes (e.g. RSpec) integrations Involves tracing integrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants