Skip to content
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

Add actioncable dependency to gemspec #515

Closed
wants to merge 1 commit into from
Closed

Conversation

xymbol
Copy link

@xymbol xymbol commented Nov 11, 2023

Turbo depends on ActionCable in runtime, but it's not listed in the gemspec. This is not a problem with the tests because of the Gemfile and, more generally, because ActionCable is a dependency of Rails.

While investigating a loading problem with turbo-rails and an app that didn't use ActionCable, I ran into this.

Turbo depends on ActionCable in runtime, but it's not listed in the gemspec.
This is not a problem with the tests because of the Gemfile and, more
generally, because ActionCable is a dependency of Rails.

While investigating a loading problem with turbo-rails and an app that didn't
use ActionCable, I ran into this.
@xymbol
Copy link
Author

xymbol commented Nov 13, 2023

Since submitting the above, I found this initializer and #512, which might have been the root cause of the FrozenError I was looking into.

config.eager_load_paths.delete("#{root}/app/channels") unless defined?(ActionCable)

I'll close this for now.

@xymbol xymbol closed this Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant