-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
Various smaller clean-up in gulpfile.js
#11869
Conversation
…ke sense Not only is there no code depending on it now, the actual task itself doesn't even make sense as-is. Note that it uses the default `DEFINES` configuration *unaltered*, which is neither useful nor correct since the resulting build thus won't make sense without an actual built target set.
…te ones for the main/worker-thread files All of the other *similar* helper functions only target one file per function, and there's no particular reason for this one to be different. This patch will simplify future changes, e.g. experimenting with using `gulp watch` instead of SystemJS for the development viewer.
cacd43e
to
a9e7798
Compare
/botio unittest |
From: Bot.io (Linux m4)ReceivedCommand cmd_unittest from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/8163f906a3b7897/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_unittest from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.215.176.217:8877/66ba5c465e0f4f6/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/8163f906a3b7897/output.txt Total script time: 3.51 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.215.176.217:8877/66ba5c465e0f4f6/output.txt Total script time: 4.60 mins
|
/botio-linux preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/3580b060de9b18d/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/3580b060de9b18d/output.txt Total script time: 3.23 mins Published |
Looks good; thanks! |
Please refer to the individual commit messages.