Skip to content

feat!: Updating to ESM and Node24#543

Open
kxc171 wants to merge 1 commit intodynatrace-oss:mainfrom
kxc171:feat/node24-esm
Open

feat!: Updating to ESM and Node24#543
kxc171 wants to merge 1 commit intodynatrace-oss:mainfrom
kxc171:feat/node24-esm

Conversation

@kxc171
Copy link
Copy Markdown
Contributor

@kxc171 kxc171 commented Mar 12, 2026

Hey, it's me again! I'm here to update this action to Node24 since Node20 will be deprecated on GitHub Runners started June 2nd, 2026. I've also incorporated some other updates as well (no functional changes, looks like more than it is due to dist/ updates).

Changes

  • Updated to Node24 [BREAKING CHANGE]
  • Updated dependencies and Actions
  • Updated to ESM format
  • Added Checkov (part of superlinter)
  • Updated Dependabot configuration to include GitHub Actions and group updates.
  • Updated linter configurations

Most of these changes align to what has been going on with GitHub's typescript-action template. Happy to make any adjustments or reduce the size of this PR too, just let me know!

Caution

This is considered a breaking change due to the bump in Node version, which requried Actions Runner version of v2.327.1 to run. This (and newer versions) have been available to public runners for many months now.

The README has been updated to reflect a v10 version because of this.

Fixes #545

@sonarqubecloud
Copy link
Copy Markdown

@kxc171
Copy link
Copy Markdown
Contributor Author

kxc171 commented Apr 2, 2026

@wolfgangB33r - Is there any chance you could review this? I'm more than happy to update this again since other PRs got merged in (surpringly), but I don't really want to waste time updating this again if you're not interested.

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@jayeshmatekar-eaton
Copy link
Copy Markdown

@kxc171 reducing PR size to segregate changes may help in getting quicker approval.
I'm also waiting for support for Nodejs.24, I could create a PR having changes only related to nodejs 24 support.

@kxc171
Copy link
Copy Markdown
Contributor Author

kxc171 commented Apr 16, 2026

@jayeshmatekar-eaton - Yeah, this something we've done before with DT - we actually updated to Node20 the same way a year ago or so haha. Figured they wouldn't mind having it spruced up again

If they want me to reduce the PR I'm more than happy to do so, I would greatly appreciate if @wolfgangB33r or whoever actively maintains this repository to say so.

I have created #546 just to try and move this along faster. There are no actual changes with Node24.

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.

Update to Node24

2 participants