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
T/473: Remove gulp dependency #602
Conversation
scripts/docs/build-docs.js
Outdated
|
||
'use strict'; | ||
|
||
const assertIsInstalled = require( './../util/assertisinstalled' ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's call this directory utils/
to be consistent.
I've just checked all the tasks. All works fine. |
@szymonkups I don't want to merge this PR and leave, but it LGTM too. I think you can merge it but please check the docs for some leftovers, cause I haven't checked that bit. |
I've checked both PR's and everything seems fine. But I will not close them: after merging PR in |
Suggested merge commit message (convention)
Other: Remove gulp dependency in favor of npm scripts. Closes #473.
BREAKING CHANGE: Gulp task are removed. New npm scripts were introduced for dev related tasks.
Additional information
./scripts/release
and./scripts/docs
) and are invoked by appropriate npmrun-script
.