Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use node 20 in CI and fix testing example #8974

Merged
merged 3 commits into from Dec 26, 2023
Merged

Conversation

borisno2
Copy link
Member

@borisno2 borisno2 commented Dec 21, 2023

We have been node --loader originally due to an issue with node 18, however with the upgrade to node 18.19 the --loader which was experimental is depreciated so tsx is throwing an error for the testing example.
This PR bumps the CI to use node 20 (latest lts) and also fixes the testing example to use --import.

Copy link

codesandbox-ci bot commented Dec 21, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 028b984:

Sandbox Source
@keystone-6/sandbox Configuration

@borisno2 borisno2 changed the title Use tsx directly for test example UP node in CI to 20 and use tsx directly for testing example Dec 21, 2023
@borisno2 borisno2 changed the title UP node in CI to 20 and use tsx directly for testing example Use node in CI to 20 and use tsx directly for testing example Dec 21, 2023
@borisno2 borisno2 changed the title Use node in CI to 20 and use tsx directly for testing example Use node 20 in CI and use tsx directly for testing example Dec 21, 2023
@borisno2 borisno2 changed the title Use node 20 in CI and use tsx directly for testing example Use node 20 in CI and fix testing example Dec 25, 2023
@dcousens dcousens merged commit 3311f44 into main Dec 26, 2023
58 checks passed
@dcousens dcousens deleted the fix-testing-example-test branch December 26, 2023 00:49
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.

None yet

2 participants