Skip to content

Move from gulp to node scripts#9006

Open
JoeRobich wants to merge 4 commits intomainfrom
dev/jorobich/remove-gulp
Open

Move from gulp to node scripts#9006
JoeRobich wants to merge 4 commits intomainfrom
dev/jorobich/remove-gulp

Conversation

@JoeRobich
Copy link
Member

@JoeRobich JoeRobich commented Feb 14, 2026

This PR moves us from Gulp to running node scripts via node run. It exposes all our tasks as entries in the packages.json scripts section. This allows us to use npm run {task} in the same manner as we were using npx gulp {task}.

Problems with gulp

  • Gulp didn't release a new version for multiple years until just recently
  • Gulp or one of the gulp tasks we relied on limited our ability to take newer versions of some dependencies
  • Also fewer dependencies is better assuming we can do it without reimplementing our own version of gulp

Aside
I investigated using node's built-in type stripping support to replace our use of ts-node, but it required changes inside the /src directory which I was trying to avoid. At some point I am sure we can move all our code to be compatible in order to avoid the compilation step. In particular the changes required meant <type> casts became as type, enum types became const lookups, and imports of Typescript type definitions were changed to import type.

@JoeRobich JoeRobich requested a review from a team as a code owner February 14, 2026 15:39
@JoeRobich JoeRobich marked this pull request as draft February 14, 2026 15:41
@JoeRobich JoeRobich force-pushed the dev/jorobich/remove-gulp branch from cb19291 to b56acb8 Compare February 14, 2026 22:38
tasks/runTask.ts Outdated
* Licensed under the MIT License. See License.txt in the project root for license information.
*--------------------------------------------------------------------------------------------*/

export function runTask(task: () => Promise<void>): void {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since CJS doesn't support top-level await we need a little wrapper function so our async tasks can run.

@JoeRobich JoeRobich force-pushed the dev/jorobich/remove-gulp branch from 7f64015 to bb44da1 Compare February 14, 2026 23:55
@JoeRobich JoeRobich marked this pull request as ready for review February 14, 2026 23:55
@JoeRobich JoeRobich requested a review from dibarbet February 17, 2026 18:38
@JoeRobich JoeRobich force-pushed the dev/jorobich/remove-gulp branch from 63eb76b to f6ef6a3 Compare February 18, 2026 05:19
@JoeRobich JoeRobich force-pushed the dev/jorobich/remove-gulp branch from 1be624a to d3741ff Compare February 18, 2026 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants