-
Notifications
You must be signed in to change notification settings - Fork 133
Open
Description
Related to #288 (cc. @tgwizard).
Request: Can we add a require 'rails/all' to the Phonelib initializer? i.e.
if defined?(Rails)
require 'rails/all' # <-- added
class Phonelib::Railtie < Rails::Railtie
Happy to make this change myself, but wanted to see if there's a faster workaround.
Context: I'm having trouble starting up the multi-process feature of Sidekiq Enterprise. More details on the issue here.
At least Rails::Railtie and Rails::Engine are undefined.
Adding gem 'phonelib', require: 'rails/all' to my Gemfile does not work.
> bundle exec sidekiqswarm
[swarm] Preloading Bundler groups ["default"]
/Users/eileenj/.rvm/gems/ruby-3.1.4/gems/phonelib-0.8.7/lib/phonelib.rb:22:in `<top (required)>': uninitialized constant Rails::Railtie (NameError)
class Phonelib::Railtie < Rails::Railtie
^^^^^^^^^
Metadata
Metadata
Assignees
Labels
No labels