Skip to content

Remove upper bound on Railsties Dependency to simplify future Rails updates#122

Open
kanejamison wants to merge 3 commits intofroala:masterfrom
ContentHarmony:master
Open

Remove upper bound on Railsties Dependency to simplify future Rails updates#122
kanejamison wants to merge 3 commits intofroala:masterfrom
ContentHarmony:master

Conversation

@kanejamison
Copy link
Copy Markdown

This PR removes the upper bounded limit on the gemspec for the railsties dependency.

The upper bound is not maintained actively on this gem to match the development pace of Rails, which means customers are forced to fork this gem and maintain their own copies in order to stay up to date on their own applications.

While there is a chance that the wysiwyg-rails gem will not be compatible with future updates to railsties - this gem will need to fix those conflicts regardless. The rest of the time, the upper bound just adds friction to Froala customers, and actively prevents us from identifying upcoming gem issues and reporting them back here.


Here is a related conversation where the Mike Perham, the author and maintainer of Sidekiq, once of the most heavily used free and paid background jobs systems for Rails, explaining why they choose not to declare upper limits in their gemspec requirements (my initial post was about another gem facing the same issue):

"This is exactly why Sidekiq only ever specifies ">=" in its gemspec version constraints. As a library in your application, I'd rather be loose and assume compatibility so I don't force unnecessary upgrades on you."

image

@kanejamison kanejamison changed the title Remove upper bound on Railstie Dependency to simplify future Rails updates Remove upper bound on Railsties Dependency to simplify future Rails updates Oct 30, 2025
@stephchuolee
Copy link
Copy Markdown

Appreciate this PR @kanejamison! Our team is hoping to upgrade to Rails 8.1 and this would unblock us. @harasunu-narayan are you be able to review and release this?

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