-
Notifications
You must be signed in to change notification settings - Fork 338
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
Refactor watch task to remove Gulp #2707
Comments
Since the watch task relies on the gulp tasks |
This issue was picked up in: It's ready for review but we'll need to decide:
|
I'm inclined to do 1 or 3.
I vote we go with 1 for now and open an issue to investigate an alternative which we can pick up if we find the solution holds us back from anything. |
Yeah, same. Now that we've decided to keeping logging via the Gulp CLI my vote goes on 1) too Mainly because there's too much overhead trying to replicate glob-watcher used by Thanks @domoscargin |
What
Update
tasks/gulp/watch.js
to replace Gulp with our own watch script. This task is responsible for watching source code changes.Why
We have made the decision to move away from Gulp and write our own build scripts.
Who needs to work on this
Developers
Who needs to review this
Developers
Done when
docs/contributing/tasks.md has been updatedThe text was updated successfully, but these errors were encountered: