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

Use overmind to run Procfiles, if available #191

Closed
wants to merge 1 commit into from

Conversation

MatheusRich
Copy link

@MatheusRich MatheusRich commented Mar 14, 2024

Overmind is a process manager (like foreman) for Procfile-based applications. The advantage is that it allows to use debuggers like pry or binding.irb without installing extra gems.


This PR is very simple, as I'm waiting for feedback on whether the maintainers like this idea. I can add tests and a CHANGELOG entry after that. I'll also open a PR on cssbundling-rails if this one is accepted.

[Overmind] is a process manager (like foreman) for Procfile-based applications. The advantage is that it [allows to use debuggers] like pry or `binding.irb` without installing extra gems.

[Overmind]: https://github.com/DarthSim/overmind
[allows to use debuggers]: https://heyvaleria.github.io/coding/2022/04/27/overmind.html
Comment on lines +6 to +8
if command -v overmind >/dev/null 2>&1; then
task_runner="overmind"
else
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add an option to opt out of this (i.e., manually picking foreman)?

@MatheusRich
Copy link
Author

@dhh thoughts?

@dhh
Copy link
Member

dhh commented Mar 14, 2024

I don't think it's worth carrying this specialization. You can just tweak it yourself after it's setup.

@dhh dhh closed this Mar 14, 2024
@semaperepelitsa
Copy link

Is there any reason to prefer Foreman vs Overmind?

@MatheusRich
Copy link
Author

MatheusRich commented Mar 15, 2024

@semaperepelitsa I believe Foreman is preferred because it's a pure Ruby solution. You can just gem install it without worrying about OSes and other packages.

@MatheusRich MatheusRich deleted the patch-1 branch March 15, 2024 14:19
@glaucocustodio
Copy link
Contributor

glaucocustodio commented Apr 5, 2024

Foreman has this annoying issue, its fork overman addresses that.

Would you accept a PR for replacing Foreman by Overman @dhh?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants