Open samikeijonen opened 3 years ago
Sorry for late reply. We don't have npm install
in the example, as it often is so slow and pulling in a lot of dependencies might cause unpredictable end results.
You are however free to modify the git hook in your own project to perfectly fit your custom needs. The project template is an example of how things can be done and there is no need to adhere to it exactly.
If/when there later are updates to the project template, you can either use git merge
or wp-fix-project
to import new upstream changes.
Yep, we already modified this :)
P.S. I'd personally add comment around gulp
since it doesn't do anything on it's own. And it makes assumption that there is no build
or dist
folder in the repo.
It's common that Gulp handles all the assets like CSS and JS. At the moment
gulp
is run when$SCSS_CHANGED
istrue
inpost-receive
hook.It would make sense that by default it would catch changes on JS files also. Maybe it can be called
ASSETS_CHANGED
instead ofSCSS_CHANGED
.And then something like:
I'm also not certain should there be
npm install
beforegulp
, haven't tested this yet.