Skip to content

Conversation

@dannon
Copy link
Member

@dannon dannon commented Nov 30, 2025

Followup to the Vite migration - replaces gulp with a plain Node.js script for
the pre-build tasks (icons and visualization plugins).

Gulp was only being used for task orchestration, so it's easy to drop in favor
of a simple script that does the same thing. One less build tool dependency to
worry about.

Adds yarn icons, yarn plugins, and yarn plugins-rebuild as shortcuts.

Regular build works the same.

Builds on #21369

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@dannon dannon force-pushed the no-more-gulp branch 3 times, most recently from 68ba7b2 to 1740859 Compare December 3, 2025 19:52
@dannon dannon marked this pull request as ready for review December 3, 2025 19:52
@dannon dannon marked this pull request as draft December 3, 2025 20:03
@dannon dannon marked this pull request as ready for review December 3, 2025 20:09
@github-actions github-actions bot added this to the 26.0 milestone Dec 3, 2025
Change build_icons.js and generate_ts_icons.js to ES modules with
.mjs extension. Uses standard ESM imports and named exports.
Simple Node.js script that handles icon building and plugin staging,
same functionality as the gulpfile but without the gulp dependency.
Switch build commands to use scripts/build.mjs and drop the gulp
dependency. Adds yarn icons, plugins, and plugins-rebuild shortcuts.
Replace gulp reference with the new Node.js build script in the
client unit test CI workflow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant