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

Remove spring-watcher-listen from default Gemfile #36377

Merged
merged 1 commit into from Dec 17, 2019

Commits on Jun 2, 2019

  1. Remove spring-watcher-listen from default Gemfile

    `spring-watcher-listen` watch application root by default.
    https://github.com/jonleighton/spring-watcher-listen/blob/c4bfe15805867e229588e4b776a7b87438137094/lib/spring/watcher/listen.rb#L58
    
    This is necessary to watch the file (e.g. `.ruby-version`) in the
    application root.
    
    By this `node_modules` also be watched, and it is a possibility to be
    shown a warning by `listen`.
    Related to rails#32700, rails#34912, rails/webpacker#1990.
    
    `listen` watches directory recursive by default, and it cannot avoid it.
    guard/listen#111
    
    So If this warning happens, the only workaround the user can do is remove
    the gem.
    
    The issue is likely to occur more frequently in Rails 6 because
    `rails new` runs `webpacker:install` by default. Because of such a
    state, I think that we should not recommend to use
    `spring-watcher-listen`.
    
    Spring has polling watcher, restart process works without this
    `spring-watcher-listen`.
    Because of polling base, CPU load may be higher than listen base. Still
    I think that it is better than the warning comes out.
    y-yagi committed Jun 2, 2019
    Configuration menu
    Copy the full SHA
    1ff98ed View commit details
    Browse the repository at this point in the history