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

Compile assets in nightly tests, and use Node 20 everywhere #50

Merged
merged 3 commits into from
Jan 19, 2024

Conversation

zerolab
Copy link
Collaborator

@zerolab zerolab commented Jan 12, 2024

Part of #22

Also standardises on Node 20 (LTS) everywhere. We had node 21 in docs and devcontainer.json, Node 16 in .nvmrc and Node 18 as default in the GitHub Actions (ref)

@zerolab zerolab mentioned this pull request Jan 12, 2024
@zerolab zerolab requested a review from tm-kn January 18, 2024 15:18
@tomusher
Copy link
Member

Should we update the devcontainer.json to match the node version too?

@zerolab
Copy link
Collaborator Author

zerolab commented Jan 18, 2024

oh, I missed that one. will do

@zerolab zerolab force-pushed the chore/compiled-assets-in-nightly-tests branch from 1e267d1 to db366dc Compare January 18, 2024 16:38
@zerolab zerolab requested review from tomusher and removed request for tm-kn January 18, 2024 16:38
@zerolab
Copy link
Collaborator Author

zerolab commented Jan 18, 2024

@tomusher done. and rebased to keep the commits clean

@zerolab zerolab changed the title Compile assets in nightly tests Compile assets in nightly tests, and use Node 20 everywhere Jan 18, 2024
@zerolab zerolab force-pushed the chore/compiled-assets-in-nightly-tests branch from db366dc to 24d6344 Compare January 18, 2024 16:51
@tomusher
Copy link
Member

Perfect thanks @zerolab !

@tomusher tomusher merged commit 3339f79 into main Jan 19, 2024
11 checks passed
@zerolab zerolab deleted the chore/compiled-assets-in-nightly-tests branch January 19, 2024 12:19
@zerolab
Copy link
Collaborator Author

zerolab commented Jan 19, 2024

Cheers @tomusher!

I think with this we're ready for a new release

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