Skip to content

Update wysiwyg-rails.gemspec#120

Closed
styx wants to merge 1 commit intofroala:masterfrom
styx:master
Closed

Update wysiwyg-rails.gemspec#120
styx wants to merge 1 commit intofroala:masterfrom
styx:master

Conversation

@styx
Copy link
Copy Markdown

@styx styx commented Jan 29, 2025

No description provided.

@kpheasey
Copy link
Copy Markdown

Any idea when this can be merged? Rails 8.0 has been out for quite some time now and official gem support would be greatly appreciated.

@kanejamison
Copy link
Copy Markdown

Also would love to see this pulled in!

@chamnap
Copy link
Copy Markdown

chamnap commented Jun 17, 2025

Would love to see this pulled in too!

@jotolo
Copy link
Copy Markdown
Contributor

jotolo commented Aug 26, 2025

Hi @harasunu-narayan , can we take a look at this one? It would be nice to get support for Rails 8. Thanks in advance

@kanejamison
Copy link
Copy Markdown

There's a good chance Rails 8.1 or similar is released at RailsWorld one month from now. It would be great to see something like gem.add_dependency "railties", ">= 3.2.7", "< 9.0" applied or better yet, to remove the upper bound altogether and let teams manage this on their own.

I know there isn't always consensus on these things but for reasons discussed in threads like this, the upper bound adds friction to all upgrades and doesn't really offer any safety to the user. Any bugs that appear with new Rails versions are going to need to be dealt with regardless - might as well have teams like us reporting them as soon as possible.

@kanejamison
Copy link
Copy Markdown

The gemspec in main now declares < 8.1 instead of < 8.0, so this PR can technically be closed.

But, I've opened a separate proposed PR that removes the upper bound entirely. I'd love to see #122 merged so that we don't have to keep going through this process each year as Rails releases new versions.

@styx styx closed this Mar 19, 2026
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.

5 participants