wrapper-validation-action icon indicating copy to clipboard operation
wrapper-validation-action copied to clipboard

Getting node 12 deprecation error from github workflows runner

Open wickkidd opened this issue 3 years ago • 3 comments

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: gradle/wrapper-validation-action

I am using v1 - uses: gradle/wrapper-validation-action@v1 and that tag seems to include the bump to node 16 from last year.

wickkidd avatar Oct 11 '22 17:10 wickkidd

@wickkidd no, v1 does not include it - there is a later commit. https://github.com/gradle/wrapper-validation-action/commit/094d8354f0cc239ee6a315e3331528fbc2008041 I think, we just need a release with latest changes.

snovikov avatar Oct 12 '22 15:10 snovikov

Ah ok. Looks like we have until summer '23 before they pull the plug on node 12.

https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Should I make a request for this or is it already planned to happen by then?

wickkidd avatar Oct 12 '22 17:10 wickkidd

We have the upgrade commit, but we don't have the release. We can use gradle/wrapper-validation-action@master at the moment.

qhy040404 avatar Oct 14 '22 11:10 qhy040404

Hey @JLLeitschuh,

we'd really appreciate it, if you could cut a release please 🙏🏻

Thanks!

NotMyFault avatar Oct 15 '22 13:10 NotMyFault

Unfortunately, I no longer work for Gradle, so I don't have the ability to do so.

CC: @big-guy, @eskatos , @pioterj

JLLeitschuh avatar Oct 18 '22 04:10 JLLeitschuh

PR cross ref: https://github.com/gradle/wrapper-validation-action/pull/53

@big-guy, @eskatos, @pioterj once you release this, please also give some love to https://github.com/gradle/gradle-build-action/issues/461.

Feels like GitHub deprecated the whole world :)

TWiStErRob avatar Oct 19 '22 18:10 TWiStErRob

@bigdaz could you please release this one too?

TWiStErRob avatar Oct 22 '22 19:10 TWiStErRob

I believe actions/core will need to be updated too

sciencewhiz avatar Oct 22 '22 22:10 sciencewhiz

@sciencewhiz what's the need for that? There's no warnings about set-output/set-state here. Here's an example of a most recent execution: https://github.com/detekt/detekt/actions/runs/3306311082

TWiStErRob avatar Oct 23 '22 09:10 TWiStErRob

I've updated a couple of dependencies (that had security vulnerabilities), and released the latest code as v1.0.5. I have not yet updated the v1 tag to point to the new release, because I'm a little concerned that I may have broken things.

Can a few folks watching this issue please test with gradle/[email protected] and report back if it's working (or not)? Once confirmed, I'll complete the release process by force-pushing the v1 tag.

bigdaz avatar Oct 24 '22 16:10 bigdaz

Can a few folks watching this issue please test with gradle/[email protected] and report back if it's working (or not)? Once confirmed, I'll complete the release process by force-pushing the v1 tag.

Hey,

thanks for the update. I can assure, that v1.0.5 works fine in my environments. This applies to highlighting wrong hashes: Screenshot 2022-10-24 at 18 46 49 and passing on correct ones: Screenshot 2022-10-24 at 18 47 12

Thanks 👏🏻

NotMyFault avatar Oct 24 '22 16:10 NotMyFault

I'm very confused about the v1.0.5 tag, it looks like it's a separate branch, whose only difference to master is an added node_modules folder, which is almost never version controlled, it's like pushing ~/.gradle/caches/files-2 to a repo. What happened there? Edit: I can see the current (2021) v1 tag is the same approach, still confused. 😅

TWiStErRob avatar Oct 24 '22 17:10 TWiStErRob

Got a success too with the new version: https://github.com/TWiStErRob/detekt/actions/runs/3314729223/jobs/5474385497#step:3:7

TWiStErRob avatar Oct 24 '22 17:10 TWiStErRob

@TWiStErRob I simply followed the documented release process. I agree that this is odd, and it's not how the gradle/gradle-build-action repository/releases are structured.

But since this is my first ever involvement with this repo, I decided to keep my changes to a minimum.

Thanks for confirming the new version works as expected. I'll force-push v1 to point to v1.0.5 shortly.

bigdaz avatar Oct 24 '22 18:10 bigdaz

Cheers @bigdaz! Thank you for picking it up here too. 🙏

@eskatos if you remember, I would be curious to know why the release process contains "push node_modules to branch",.

TWiStErRob avatar Oct 24 '22 18:10 TWiStErRob

This action doesn't get minified/assembled and runs directly from node_modules

eskatos avatar Oct 25 '22 06:10 eskatos