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

Add Puma plugin #64

Closed

Conversation

jacob-carlborg-apoex
Copy link
Contributor

This manages the Dartsass watch process from rails server.

Most of this is taken from https://github.com/rails/tailwindcss-rails.

This manages the Dartsass watch process from `rails server`
@northeastprince
Copy link

@dhh could we get this merged?

@dhh
Copy link
Member

dhh commented Nov 25, 2024

We've doubled down on bin/dev using foreman to do this. Appreciate the work, but I don't think there's enough of a benefit from adding this to warrant the confusion of when you should use this vs another.

@dhh dhh closed this Nov 25, 2024
@northeastprince
Copy link

There doesn't seem to be a way to do that with puma-dev

@dhh
Copy link
Member

dhh commented Nov 25, 2024

I wouldn't recommend using that anymore anyway. You can use localhost as a secure domain and just go off the ports. That's what we do at 37signals.

@jacob-carlborg-apoex
Copy link
Contributor Author

We've doubled down on bin/dev using foreman to do this

I prefer using Puma plugins in favor of Foreman. Then I don't need to use remote debugging to set a breakpoint.

@jacob-carlborg-apoex
Copy link
Contributor Author

@northeastprince you can just copy this plugin to your application if you want to use it anyway. I've already done that. Just inline the dartsass_watch_command method at line 15 in the plugin, then you only need the file with the plugin.

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.

3 participants