exception_notification icon indicating copy to clipboard operation
exception_notification copied to clipboard

Rails 8 support

Open andynu opened this issue 1 year ago • 13 comments

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:

andynu avatar Nov 08 '24 15:11 andynu

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.

brandonzylstra avatar Nov 19 '24 01:11 brandonzylstra

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

kmcphillips avatar Dec 03 '24 03:12 kmcphillips

This is on the way to being resolved 🎉

https://github.com/smartinez87/exception_notification/pull/544

Thanks @smartinez87

Petercopter avatar Dec 04 '24 19:12 Petercopter

This is on the way to being resolved 🎉

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

brandonzylstra avatar Dec 04 '24 21:12 brandonzylstra

@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"

RomainMorlevat avatar Dec 17 '24 16:12 RomainMorlevat

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

SebastianPoell avatar Dec 30 '24 11:12 SebastianPoell

@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!

markets avatar Jan 17 '25 11:01 markets

@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.

kmcphillips avatar Jan 21 '25 18:01 kmcphillips

@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'

akaspick avatar Jan 24 '25 16:01 akaspick

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.

kmcphillips avatar Jan 24 '25 16:01 kmcphillips

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.

akaspick avatar Jan 24 '25 16:01 akaspick

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.

SebastianPoell avatar Jan 24 '25 16:01 SebastianPoell

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

akaspick avatar Jan 24 '25 16:01 akaspick

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:

  • https://github.com/smartinez87/exception_notification/pull/551

kmcphillips avatar Mar 20 '25 18:03 kmcphillips