ronin-rb / ronin-app

A local web interface for Ronin.
https://ronin-rb.dev
GNU Affero General Public License v3.0
6 stars 6 forks source link

Deprecated `Sinatra::Reloader` #100

Open AI-Mozi opened 3 months ago

AI-Mozi commented 3 months ago

I think we should consider changing Sinatra::Reloader to an alternative like rerun or rack-unreloader because it's deprecated.

postmodern commented 3 months ago

Mastodon recommends rerun or entr; although I can't find any info on how to use entr to reload sinatra. @AI-Mozi you're call.

postmodern commented 3 months ago

According to this toot, we could just ass rerun or entr to the Procfile.dev and tell it what files to watch and it will kill/re-execute the command if any of them change. Seems kind of heavy handed though, I would prefer an in-code reloader that didn't re-start the whole app and web server.