Skip to content

[VAULT-39456] upgrade to node 24#63

Merged
ryancragun merged 3 commits into
mainfrom
ryan/VAULT-39456
Sep 11, 2025
Merged

[VAULT-39456] upgrade to node 24#63
ryancragun merged 3 commits into
mainfrom
ryan/VAULT-39456

Conversation

@ryancragun

@ryancragun ryancragun commented Sep 10, 2025

Copy link
Copy Markdown
Contributor

How to read this pull request

Upgrade our github action to run on Node 24. As part of this we upgrade
to modern deps and migrate away from jest.

  • Update PR template
  • Migrate from a CommonJS to ES Modules
  • Upgrade to Node 24
  • Upgrade eslint to v9
  • Migrate from jest to vitest
  • Update default version to enos 0.0.33

Signed-off-by: Ryan Cragun me@ryan.ec

Checklist

  • The commit message includes an explanation of the changes
  • Manual validation of the changes have been performed (if possible)
  • New or modified code has requisite test coverage (if possible)
  • I have performed a self-review of the changes
  • I have made necessary changes and/or pull requests for documentation
  • I have written useful comments in the code

compliance-pr-automation[bot] and others added 2 commits September 10, 2025 10:10
@ryancragun ryancragun requested a review from a team as a code owner September 10, 2025 21:53
@ryancragun ryancragun force-pushed the ryan/VAULT-39456 branch 3 times, most recently from 105f3f1 to f51dcc2 Compare September 10, 2025 22:04
Comment thread dist/sourcemap-register.cjs Fixed
raskchanky
raskchanky previously approved these changes Sep 10, 2025

@raskchanky raskchanky 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.

I'm assuming the dist/* files are generated so I didn't review those by hand, esp not the 40K line diff. If I'm wrong, let me know.

Upgrade our github action to run on Node 24. As part of this we upgrade
to modern deps and migrate away from jest.

* Migrate from a CommonJS module to ES Modules
* Upgrade to Node 24
* Upgrade eslint to v9
* Migrate from jest to vitest
* Update default version to enos 0.0.33

Signed-off-by: Ryan Cragun <me@ryan.ec>
@ryancragun ryancragun merged commit 39a8f38 into main Sep 11, 2025
8 checks passed
@ryancragun ryancragun deleted the ryan/VAULT-39456 branch September 11, 2025 16:56
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.

3 participants