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

chore: Update Node #799

Merged
merged 4 commits into from
Sep 14, 2023
Merged

chore: Update Node #799

merged 4 commits into from
Sep 14, 2023

Conversation

vsgoulart
Copy link
Contributor

@pinussilvestrus
Copy link
Contributor

@Skaiir @vsgoulart I was able to fix the three test cases by using act. It could be that bigger forms take longer in some environments to import so it makes sense to wait for it.

@pinussilvestrus
Copy link
Contributor

pinging @bpmn-io/modeling-dev that we're targetting Node@18 now for our CI, with the reasons mentioned above (End of Life).

@nikku
Copy link
Member

nikku commented Sep 14, 2023

Any reason you don't switch to Node@20 right away? I guess that is what we'd attempt to do, to safe some effort in 1 year.

@nikku
Copy link
Member

nikku commented Sep 14, 2023

Created a tracking issue on our end, thanks for the heads-up @pinussilvestrus 🏅

@pinussilvestrus
Copy link
Contributor

Any reason you don't switch to Node@20 right away? I guess that is what we'd attempt to do, to safe some effort in 1 year.

No real reason, we will go to 20 directly in case we see no blockers 👍 thanks!

@pinussilvestrus pinussilvestrus merged commit 87ced7a into develop Sep 14, 2023
6 of 10 checks passed
@pinussilvestrus pinussilvestrus deleted the update-node branch September 14, 2023 09:33
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Sep 14, 2023
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