From e602358dd5fcd8fda175fb484705e4098580882c Mon Sep 17 00:00:00 2001 From: Christopher Allen Date: Mon, 26 Sep 2022 17:18:41 +0200 Subject: [PATCH] feat(build): Make cleanup, license [sic] tasks invoke their prerequisites Turns out they don't have any, so this commit just classifies their gulp targets according to the established scheme. --- gulpfile.js | 6 +++--- scripts/gulpfiles/cleanup_tasks.js | 1 + scripts/gulpfiles/license_tasks.js | 1 + 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/gulpfile.js b/gulpfile.js index 838269577..83ed2709a 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -37,10 +37,12 @@ module.exports = { publishBeta: releaseTasks.publishBeta, gitUpdateGithubPages: gitTasks.updateGithubPages, - // Manually-invokable targets that invoke prerequisites. + // Manually-invokable targets, with prequisites where required. prepare: buildTasks.prepare, format: buildTasks.format, generate: buildTasks.generate, + sortRequires: cleanupTasks.sortRequires, + checkLicenses: licenseTasks.checkLicenses, clean: gulp.parallel(buildTasks.cleanBuildDir, packageTasks.cleanReleaseDir), buildAdvancedCompilationTest: buildTasks.buildAdvancedCompilationTest, gitCreateRC: gitTasks.createRC, @@ -57,7 +59,5 @@ module.exports = { // TBD. deployDemos: appengineTasks.deployDemos, deployDemosBeta: appengineTasks.deployDemosBeta, - checkLicenses: licenseTasks.checkLicenses, prepareDemos: appengineTasks.prepareDemos, - sortRequires: cleanupTasks.sortRequires, }; diff --git a/scripts/gulpfiles/cleanup_tasks.js b/scripts/gulpfiles/cleanup_tasks.js index e6c8501ff..5b4a933fb 100644 --- a/scripts/gulpfiles/cleanup_tasks.js +++ b/scripts/gulpfiles/cleanup_tasks.js @@ -81,5 +81,6 @@ function sortRequires() { }; module.exports = { + // Manually-invokable targets, with prequisites where required. sortRequires: sortRequires }; diff --git a/scripts/gulpfiles/license_tasks.js b/scripts/gulpfiles/license_tasks.js index 3e6b93efb..8e2a4017d 100644 --- a/scripts/gulpfiles/license_tasks.js +++ b/scripts/gulpfiles/license_tasks.js @@ -17,5 +17,6 @@ function checkLicenses() { }; module.exports = { + // Manually-invokable targets, with prequisites where required. checkLicenses: checkLicenses };