New issue

Jump to bottom

feat!: require node v22.13 #34524

Draft

viceice wants to merge 1 commit into main from feat/node-v22

Member

viceice commented Feb 27, 2025

Changes

Require Node v22.13 for cjs-esm compabillity

Context

Documentation (please check one with an [x])

I have updated the documentation, or
No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

Code inspection only, or
Newly added/modified unit tests, or
No unit tests but ran on a real repository, or
Both unit tests + ran on a real repository


          feat!: require node v22.13

3b75c81

viceice added the breaking label

viceice added this to the v40 milestone

Contributor

trim21 commented Mar 1, 2025

I think we do not actually this in production? This only affect development env for running tests and can be fixed by rename one ts file to mts file?

Member Author

viceice commented Mar 1, 2025

no, we need it to allow require esm only dependencies. see next branch for samples

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels