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

feat: relax rack-test dependency to allow version 2 #270

Merged
merged 1 commit into from
Sep 28, 2022

Conversation

stanhu
Copy link
Contributor

@stanhu stanhu commented Aug 19, 2022

rack-test has a number of fixes and improvements:
https://github.com/rack/rack-test/blob/main/History.md

@bethesque
Copy link
Member

See #271

@bethesque
Copy link
Member

Sorry, see #268

@stanhu
Copy link
Contributor Author

stanhu commented Aug 24, 2022

As far as I can tell, there shouldn't be any issues with this update: #268 (comment)

@bethesque
Copy link
Member

What I'll do is open up the version for ruby, but lock it for the pact-ruby-standalone, so if there are any impacts, the ruby people will find them first!

@lsylvester
Copy link

For what it is worth I have implemented a producer and consumer using rack-test 2.X from my branch and didn't encounter any issues.

@bethesque
Copy link
Member

Locked rack-test in pact-ruby-standalone pact-foundation/pact-ruby-standalone@3fc12c9

@bethesque
Copy link
Member

Also in pact-cli pact-foundation/pact-ruby-cli@b558532

@bethesque bethesque merged commit a619deb into pact-foundation:master Sep 28, 2022
@bethesque
Copy link
Member

New version of pact gem released with this change.

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