Skip to content

Rails 8 support #545

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

Closed
andynu opened this issue Nov 8, 2024 · 14 comments
Closed

Rails 8 support #545

andynu opened this issue Nov 8, 2024 · 14 comments

Comments

@andynu
Copy link

andynu commented Nov 8, 2024

Steps to reproduce

Make a rails project with:

gem 'rails', '~> 8.0.0'
gem 'exception_notification', '~> 4.5.0'

Attempt to bundle.

Expected behavior

success.

Actual behavior

failure:

Fetching gem metadata from https://rubygems.org/........
Resolving dependencies...
Could not find compatible versions

Because exception_notification >= 4.5.0 depends on activesupport >= 5.2, < 8
  and rails >= 8.0.0 depends on activesupport = 8.0.0,
  exception_notification >= 4.5.0 is incompatible with rails >= 8.0.0.
So, because Gemfile depends on rails ~> 8.0.0
  and Gemfile depends on exception_notification ~> 4.5.0,
  version solving has failed.

System configuration

Rails version: 8.0.0

Ruby version: 3.3.5

Other configurations:

@brandondrew
Copy link

The strange thing is that version 4.1.1 claims to support Rails 8, but this is rather doubtful. It appears more likely that its lack of an upper limit was an oversight.

@kmcphillips
Copy link
Collaborator

Yeah it downgraded to the last version that didn't have a configured upper limit.

@Petercopter
Copy link

This is on the way to being resolved 🎉

#544

Thanks @smartinez87

@brandondrew
Copy link

This is on the way to being resolved 🎉

@Petercopter
On the way? Or already resolved (if we use the master branch)?

@RomainMorlevat
Copy link

@Petercopter On the way? Or already resolved (if we use the master branch)?

Using that on Gemfile does the job:
gem "exception_notification", git: "https://github.com/smartinez87/exception_notification.git", branch: "master"

@SebastianPoell
Copy link

SebastianPoell commented Dec 30, 2024

Yeah, a new release would be awesome (last was Jan 2022) :)

@markets
Copy link

markets commented Jan 17, 2025

@smartinez87 Sorry for the ping 🙏🏼 but could you please release a new version? It will be really useful for people upgrading to Rails v8. I know we can pin the gem to GitHub master branch, but a new release in RubyGems will make the upgrade more simple for everybody.

Thanks!

@kmcphillips
Copy link
Collaborator

@smartinez87 @FLarra @mschnitzer All of you have published this gem before. Would it be possible to please do a release. It blocks Rails upgrades and hasn't had a release in years. Just push what's on main.

@akaspick
Copy link

akaspick commented Jan 24, 2025

@kmcphillips Just do what @RomainMorlevat mentioned.... point to the main github branch in your Gemfile to get the current updates.

My Gemfile has gem 'exception_notification', github: 'smartinez87/exception_notification'

@kmcphillips
Copy link
Collaborator

I understand that, but everyone who uses this gem shouldn't have to figure out how to do that, just because the maintainers won't cut a release. This last version has been downloaded 4 million times.

@akaspick
Copy link

Most people are likely still not on Rails 8 and the gem is working fine for them still.

In a perfect world, a new version of the gem would have been released, but it's an open source project, so anyone who can't install the gem with rail 8 will just end up here eventually with the above solution in the mean time.

At least the main branch has the fix. A lot of time, I've had to refer to a completely different fork or fork my own branch of gems to get compatibility working.

@SebastianPoell
Copy link

SebastianPoell commented Jan 24, 2025

Note, that simply adding github to that line will not lead to a stable solution - you are just pulling whatever is currently on main/master - which might change any day. You might end up with different versions on different machines. Add an additional ref to that line in your Gemfile to pin one particular commit.

@akaspick
Copy link

@SebastianPoell Correct, so gem 'exception_notification', github: 'smartinez87/exception_notification', ref: '26441fb' is a better solution.

@kmcphillips
Copy link
Collaborator

Alright I got collaborator access and rubygems push access.

Anyone who wants to have a look at this PR, it gets the build green and fixes enough I think to push a new version. Feedback welcome:

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.

8 participants