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.

Sign up for GitHub

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

Jump to bottom

Deprecated Sinatra::Reloader #100

Open
AI-Mozi opened this issue Apr 20, 2024 · 2 comments
Open

Deprecated Sinatra::Reloader #100

AI-Mozi opened this issue Apr 20, 2024 · 2 comments
Assignees
@AI-Mozi
Labels
chore You gotta do, what you gotta do enhancement New feature or request help-wanted Extra attention is needed

Comments

@AI-Mozi
Copy link
Member

AI-Mozi commented Apr 20, 2024

I think we should consider changing Sinatra::Reloader to an alternative like rerun or rack-unreloader because it's deprecated.
@postmodern postmodern added chore You gotta do, what you gotta do enhancement New feature or request labels Apr 21, 2024
@postmodern
Copy link
Member

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
Copy link
Member

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.

@postmodern postmodern added the help-wanted Extra attention is needed label Aug 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AI-Mozi AI-Mozi

Labels
chore You gotta do, what you gotta do enhancement New feature or request help-wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@postmodern @AI-Mozi