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

fix(node16): Use action/checkout v4 as Node 16 is deprecated #26

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

julienloizelet
Copy link
Collaborator

The Issue

There are warning messages (complaining about Node 16 deprecation) when using this action.
Capture d’écran du 2024-02-11 16-25-43

How This PR Solves The Issue

We are updating action/checkout from v3 to v4.

For the Homebrew action, we should just wait for their update:

Homebrew/actions#484

That's why I'm creating this PR as a draft and will change it to "ready for review" when the Homebrew update is available.

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@rfay
Copy link
Member

rfay commented Feb 19, 2024

Any reason not to go forward with this?

@julienloizelet
Copy link
Collaborator Author

Any reason not to go forward with this?

Well, no good reason, I guess.

We have 2 "Node 16 deprecated" related errors: 1 for the action/checkout action and one for the homebrew/action.

On the homebrew side, we can't do nothing; and I was waiting for their update: Homebrew/actions#484

So, even if we merge this PR, we will still have an error for the homebrew action.

But, as we can't do nothing more, we could definitely merge this one to remove the "action/checkout" deprecation message.

I pass this PR in "ready for review".

Thanks

@julienloizelet julienloizelet marked this pull request as ready for review February 19, 2024 02:51
Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

Seems OK to me. Use your own best judgement. Thanks!

@julienloizelet julienloizelet merged commit 69591e9 into main Feb 19, 2024
21 checks passed
@julienloizelet julienloizelet deleted the feat/node-16-deprecated-message branch May 24, 2024 05:15
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