We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The newest release of Spring (4.1.2) and particularly this commit makes this watcher broken:
NoMethodError: undefined method `synchronize' for #<Spring::Watcher::Listen:0x00000001062a2e50 ... ... /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/spring-watcher-listen-2.1.0/lib/spring/watcher/listen.rb:61:in `changed' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/event/config.rb:28:in `call' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/event/processor.rb:117:in `block in _process_changes' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/thread.rb:26:in `rescue_and_log' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/event/processor.rb:116:in `_process_changes' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/event/processor.rb:25:in `block in loop_for' /Users/thierry/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/listen-3.8.0/lib/listen/event/processor.rb:20:in `loop' ...
I believe this call to synchronize should be changed to @mutex.synchronize to mimic what happened in that commit
synchronize
@mutex.synchronize
The text was updated successfully, but these errors were encountered:
Bring back Spring::Watcher::Abstract#synchronize
Spring::Watcher::Abstract#synchronize
c1facc5
Fix: rails/spring-watcher-listen#36
I just released Spring 4.1.3 which should fix this issue.
Sorry, something went wrong.
Thank you very much for the reactivity 🙏
Successfully merging a pull request may close this issue.
The newest release of Spring (4.1.2) and particularly this commit makes this watcher broken:
I believe this call to
synchronize
should be changed to@mutex.synchronize
to mimic what happened in that commitThe text was updated successfully, but these errors were encountered: