Skip to content

Conversation

@jhlegarreta
Copy link
Member

Fix Node.js warnings linked to GitHub actions: transition to actions/checkout@v3, and actions/setup-python@v4.

Fixes:

Node.js 12 actions are deprecated. For more information see:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.
Please update the following actions to use Node.js 16:
actions/checkout@v2, actions/setup-python@v2, lukka/get-cmake@v3.22.2

raised for example in:
https://github.com/InsightSoftwareConsortium/ITKIOMeshSTL/actions/runs/3791396544

Fix `Node.js` warnings linked to GitHub actions: transition to
`actions/checkout@v3`, and `actions/setup-python@v4`.

Fixes:
```
Node.js 12 actions are deprecated. For more information see:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.
Please update the following actions to use Node.js 16:
actions/checkout@v2, actions/setup-python@v2, lukka/get-cmake@v3.22.2
```

raised for example in:
https://github.com/InsightSoftwareConsortium/ITKIOMeshSTL/actions/runs/3791396544
Copy link
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

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

@jhlegarreta thanks!

Copy link
Collaborator

@tbirdso tbirdso left a comment

Choose a reason for hiding this comment

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

Thanks @jhlegarreta !

@tbirdso tbirdso merged commit 15371f5 into InsightSoftwareConsortium:main Dec 29, 2022
@jhlegarreta jhlegarreta deleted the FixGitHubActionsNodejsWarning branch December 29, 2022 17:51
@jhlegarreta
Copy link
Member Author

Looks like this did not solve all Node.js warnings:
https://github.com/InsightSoftwareConsortium/ITKBSplineGradient/actions/runs/3824095979

The ones related to the lukka/get-cmake@v3.22.2 action are still present.

We may need to transition to lukka/get-cmake@v3.24.2: cross-referencing lukka/get-cmake#46. I will propose a PR.

@jhlegarreta
Copy link
Member Author

We may need to transition to lukka/get-cmake@v3.24.2: cross-referencing lukka/get-cmake#46. I will propose a PR.

PR #44.

@jhlegarreta
Copy link
Member Author

PR #45 🤦.

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.

4 participants