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

[WiP] Update node version to 16 in github ci #684

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

guergana
Copy link
Contributor

@guergana guergana commented Aug 14, 2023

Checks are failing at this step in the checks. This error was probably introduced when upgrading github actions that don't support node 12 in this commit.

Upgrading to node 16 works partially but now tests are complaing about the deprecation of lerna bootstrap.

@github-actions
Copy link

Copy link
Collaborator

@micgro42 micgro42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good so far! While this is probably the most important of the files, could you change it in the other workflows (I think pre-release.yaml and release.yaml) as well?

@guergana
Copy link
Contributor Author

oh my, i had changed it but forgot to save the files. 🙈

@guergana guergana changed the title Update node version to 16 in github ci [WiP] Update node version to 16 in github ci Aug 14, 2023
@guergana guergana marked this pull request as draft August 14, 2023 14:23
micgro42 added a commit that referenced this pull request Aug 15, 2023
If that is not explicitly set, CI will use whatever comes with
`ubuntu_latest` and that is not what we want for this workflow.
Node 14 is consistent with what is used in the other workflows. The PR
to upgrade these is #684.
micgro42 added a commit that referenced this pull request Aug 16, 2023
If that is not explicitly set, CI will use whatever comes with
`ubuntu_latest` and that is not what we want for this workflow.
Node 14 is consistent with what is used in the other workflows. The PR
to upgrade these is #684.
micgro42 added a commit that referenced this pull request Aug 16, 2023
If that is not explicitly set, CI will use whatever comes with `ubuntu_latest` and that is not what we want for this workflow. Node 14 is consistent with what is used in the other workflows. The PR to upgrade these is #684.

This should be merged after #688 and make CI green again on merged branches.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants