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

Question: Usage of rails-erb-loader #216

Closed
rhys-vdw opened this issue Apr 1, 2017 · 9 comments
Closed

Question: Usage of rails-erb-loader #216

rhys-vdw opened this issue Apr 1, 2017 · 9 comments

Comments

@rhys-vdw
Copy link

rhys-vdw commented Apr 1, 2017

Hey maintainers, just had a question about your usage of rails-erb-loader. (I'm not a consumer of this project, but I am the maintainer of rails-erb-loader).

Why disable spring in your runner config option? The loader spins up a new instance of rails in a child process for every .erb file fed into it. I would expect this to make rails-erb-loader grind to an intolerably slow speed.

@matthewd
Copy link
Member

matthewd commented Apr 1, 2017

@gauravtiwari

@ytbryan
Copy link
Contributor

ytbryan commented Apr 1, 2017

It's first added here #66 cc @stevejackson

@rhys-vdw
Copy link
Author

rhys-vdw commented Apr 1, 2017

Thanks @ytbryan for the info.

rails-erb-loader also seems to have problems with spring, so I've just disabled it.

- from #66 (comment)

@stevejackson I wonder if you can remember the problems you were hitting? If so could you open an issue in the rails-erb-loader repo with the details.

@ytbryan
Copy link
Contributor

ytbryan commented Apr 2, 2017

@rhys-vdw please take a look at this #220

@stevejackson
Copy link
Contributor

@rhys-vdw

It was me who wrote that, sorry for the slow reply. It was likely a mistake on my part - I was testing out webpacker in RAILS_ENV=production, which doesn't have spring installed - therefore it couldn't find the spring executable.

We could probably remove this DISABLE_SPRING=1.

@rhys-vdw
Copy link
Author

rhys-vdw commented Apr 6, 2017

We could probably remove this DISABLE_SPRING=1.

Great. Well then I would suggest removing it then.

@stevejackson
Copy link
Contributor

Will do - I'll take care of this soon.

@gauravtiwari
Copy link
Member

Feel free to make a PR for this 😄

@gauravtiwari
Copy link
Member

Fixed by #256

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

No branches or pull requests

5 participants