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

Rails 8 Compatibility (PR attached) #589

Closed
tagliala opened this issue Nov 10, 2024 · 3 comments Β· Fixed by #590
Closed

Rails 8 Compatibility (PR attached) #589

tagliala opened this issue Nov 10, 2024 · 3 comments Β· Fixed by #590

Comments

@tagliala
Copy link
Contributor

πŸ€” What's the problem you're trying to solve?

Upgrading to Rails 8.0

✨ What's your proposed solution?

Update gemspec

⛏ Have you considered any alternatives or workarounds?

No response

πŸ“š Any additional context?

No response

tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 10, 2024
- Conditionally load the sqlite3 gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

Close cucumber#589
tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 10, 2024
- Conditionally load the sqlite3 gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

Close cucumber#589
tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 10, 2024
- Conditionally load the sqlite3 gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

Close cucumber#589
@tagliala tagliala mentioned this issue Nov 10, 2024
7 tasks
@tagliala tagliala changed the title Rails 8 Compatibility Rails 8 Compatibility (PR attached) Nov 10, 2024
@luke-hill
Copy link
Contributor

Will look to get on this in the next few days @tagliala

tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 12, 2024
- Conditionally load the sqlite3 gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

Close cucumber#589
tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 13, 2024
In specific:
- Do not test Rails 7.0 against 3.3
- Allow 8.0 in gemspec, by relaxing the upper constraint to `< 8.1`.
  Rails uses shifted SemVer, so `8.1` is considered a Major.
- Conditionally load the sqlite3 gem based on the Rails version.
- Conditionally load selenium webdriver gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

References:
- https://guides.rubyonrails.org/maintenance_policy.html#versioning
- rails/rails#53045

Close cucumber#589
tagliala added a commit to tagliala/cucumber-rails that referenced this issue Nov 21, 2024
In specific:
- Do not test Rails 7.0 and 7.1 against 3.3
- Allow 8.0 in gemspec, by relaxing the upper constraint to `< 9`.
  Rails uses shifted SemVer, so `8.1` is considered a Major, but it
  has been decided to use `< 9` anyway
- Conditionally load the sqlite3 gem based on the Rails version.
- Conditionally load selenium webdriver gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

References:
- https://guides.rubyonrails.org/maintenance_policy.html#versioning
- rails/rails#53045

Close cucumber#589
@mgmodell
Copy link

Any updates on this or when it will be posted? Is it a time issue or are there compatibility issues?

@luke-hill
Copy link
Contributor

It'll be merged and a release will be cut soon

luke-hill pushed a commit that referenced this issue Nov 21, 2024
In specific:
- Do not test Rails 7.0 and 7.1 against 3.3
- Allow 8.0 in gemspec, by relaxing the upper constraint to `< 9`.
  Rails uses shifted SemVer, so `8.1` is considered a Major, but it
  has been decided to use `< 9` anyway
- Conditionally load the sqlite3 gem based on the Rails version.
- Conditionally load selenium webdriver gem based on the Rails version.
- Simplify error page assertions to account for variations in
  apostrophes, as "We're sorry" changed to "We`re sorry" in Rails 8.

References:
- https://guides.rubyonrails.org/maintenance_policy.html#versioning
- rails/rails#53045

Close #589
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 a pull request may close this issue.

3 participants