Skip to content

Bundle latest packages in distribution - #80

Merged
wawe merged 3 commits into
mainfrom
update-distribution
Jun 30, 2026
Merged

Bundle latest packages in distribution#80
wawe merged 3 commits into
mainfrom
update-distribution

Conversation

@wawe

@wawe wawe commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

During the last months, we forgot to update the actual distribution (dist/index.js) when performing package updates, because we did not update the workflow file when switching to ncc as bundler.

This PR alleviates that by

  • checking the source against the distribution in CI again
  • Switching to ESM syntax (because @actions/core 3.0.0 requires this)
  • Re-building distribution file with ncc

@wawe
wawe requested a review from a team as a code owner June 25, 2026 08:59
@wawe wawe changed the title Compare distribution with source in CI Bundle latest packages in distribution Jun 25, 2026
wawe added 2 commits June 25, 2026 16:03
Every time a dependency is updated, we also need to rebuild the bundled distribution (dist/index.js). To force this, we rebuild the distribution in CI and make sure that there are no changes.
Staring with 3.0.0, CJS is no longer supported by @actions/core.
Therefore, all imports and exports need to use ESM module syntax.
@wawe
wawe force-pushed the update-distribution branch from 791e1ee to 2965d80 Compare June 25, 2026 14:08
Update dist/index.js to match the source files.

@maltinabasse maltinabasse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@wawe
wawe merged commit b595a88 into main Jun 30, 2026
2 checks passed
@wawe
wawe deleted the update-distribution branch June 30, 2026 08:10
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