Skip to content

Bump node version to 24 and update dependencies.#36

Merged
de-vri-es merged 4 commits into
mainfrom
node24
Apr 17, 2026
Merged

Bump node version to 24 and update dependencies.#36
de-vri-es merged 4 commits into
mainfrom
node24

Conversation

@de-vri-es

Copy link
Copy Markdown
Owner

No description provided.

@de-vri-es de-vri-es requested a review from Copilot April 17, 2026 09:45

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates this GitHub Action’s runtime/entrypoint to a newer Node version and switches the shipped action code from CommonJS to ESM.

Changes:

  • Switch action entrypoint from lib/main.js (CJS) to lib/main.mjs (ESM) and update config accordingly
  • Improve credential handling by masking secrets and avoiding duplicate credential entries
  • Update documentation to reflect the new Node runtime compatibility guidance

Reviewed changes

Copilot reviewed 4 out of 626 changed files in this pull request and generated 5 comments.

File Description
lib/main.mjs New ESM entrypoint; adds duplicate avoidance + masking; includes bundled sourcemap
lib/main.js Removes old CommonJS build artifact previously used by the action
action.yml Updates runtime (using) and points main to the new .mjs entrypoint
README.md Updates compatibility guidance to reference the new Node runtime and fallback version

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread action.yml
Comment thread lib/main.mjs
Comment thread lib/main.mjs
Comment thread lib/main.mjs
Comment thread lib/main.mjs
@de-vri-es de-vri-es merged commit 5dd446b into main Apr 17, 2026
9 checks passed
@de-vri-es de-vri-es deleted the node24 branch April 17, 2026 11:07
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