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

Loosen rails version constraint in v5 & v6 fixtures #656

Merged
merged 2 commits into from
Mar 29, 2021

Conversation

imjoehaines
Copy link
Contributor

@imjoehaines imjoehaines commented Mar 26, 2021

Goal

Loosens the version constraint for rails 5 & 6 fixtures from ~> x.0.0 to ~> x. This was inadvertently stopping us from testing on the latest v5/v6 versions as we were locked to 5.0.x/6.0.x — for example, we weren't test against Rails 6.1

Testing

Covered by CI, requires #655 to pass

@imjoehaines imjoehaines removed the request for review from Cawllec March 26, 2021 17:13
@imjoehaines imjoehaines force-pushed the include-connection-id-in-breadcrumbs branch 2 times, most recently from 9266668 to c2eab54 Compare March 26, 2021 17:50
This was inadvertently stopping us from testing on the latest v5/v6
versions as we were locked to 5.0.x/6.0.x
@imjoehaines imjoehaines force-pushed the loosen-rails-version-constraint branch from 4804979 to d364484 Compare March 26, 2021 18:06
Rails 5.2 removed this method - we don't need it for anything so can
remove the call without further changes
Base automatically changed from include-connection-id-in-breadcrumbs to next March 26, 2021 18:27
@imjoehaines imjoehaines merged commit 1853b69 into next Mar 29, 2021
@imjoehaines imjoehaines deleted the loosen-rails-version-constraint branch March 29, 2021 08:04
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.

2 participants