Skip to content

Use overmind to run Procfiles, if available#191

Closed
MatheusRich wants to merge 1 commit into
rails:mainfrom
MatheusRich:patch-1
Closed

Use overmind to run Procfiles, if available#191
MatheusRich wants to merge 1 commit into
rails:mainfrom
MatheusRich:patch-1

Conversation

@MatheusRich
Copy link
Copy Markdown

@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 thread lib/install/dev
Comment on lines +6 to +8
if command -v overmind >/dev/null 2>&1; then
task_runner="overmind"
else
Copy link
Copy Markdown
Author

@MatheusRich MatheusRich Mar 14, 2024

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
Copy Markdown
Author

@dhh thoughts?

@dhh
Copy link
Copy Markdown
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
Copy Markdown

Is there any reason to prefer Foreman vs Overmind?

@MatheusRich
Copy link
Copy Markdown
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
Copy Markdown
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.

4 participants