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

Remove the hot file from versionning #4121

Merged
merged 1 commit into from Jan 27, 2017
Merged

Remove the hot file from versionning #4121

merged 1 commit into from Jan 27, 2017

Conversation

jpmurray
Copy link
Contributor

@jpmurray jpmurray commented Jan 27, 2017

Following laravel/framework#17571, I was correctly pointed that the small proposition should be made over here. Here's a copy / past of my message, for clarity's sake.


When you run npm run hmr, a hot file is created to the public folder for the mix() helper to know if it has to server from the webpack server or not. This file does not go away until we run npm run dev (or watch).

True, if we use npm run production in our production server it goes away too, but in case we don't make it go away in development, I don't think it's necessary to version it.

Following laravel/framework#17571, I was correctly pointed that the small proposition should be made over here. Here's a copy / past of my message, for clarity's sake.
---

When you run `npm run hmr`, a `hot` file is created to the public folder for the `mix()` helper to know if it has to server from the webpack server or not. This file does not go away until we run `npm run dev` (or `watch`). 

True, if we use `npm run production` in our production server it goes away too, but in case we don't make it go away in development, I don't think it's necessary to version it.
@taylorotwell taylorotwell merged commit 547bb96 into laravel:master Jan 27, 2017
appleboy added a commit to appleboy/gitignore that referenced this pull request Jan 27, 2017
@jpmurray jpmurray deleted the patch-1 branch January 27, 2017 12:02
bkrukowski pushed a commit to bkrukowski/laravel that referenced this pull request Mar 10, 2017
Remove the hot file from versionning
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants