-
Notifications
You must be signed in to change notification settings - Fork 207
should use node 16, as node 12 is depreciated #87
Comments
Can use this in the Install node step, right?
|
no the problem is not on the github action you create with your project but inside this github action, it simply does not work anymore until it is updated to node 16 |
@samuelmeuli please please fix this (:
|
@samuelmeuli Yes, my project literally depends on this for releases. Please update it. |
Just update the node version in the yml file from 10 to 16 and things will work for you, instead of waiting for the maintainer to fix it |
Is it https://github.com/samuelmeuli/action-electron-builder/blob/master/action.yml#L56 that should be changed? |
@petervanderwalt No, in the yml file that you put in your |
I have, see https://github.com/OpenBuilds/OpenBuilds-CONTROL/blob/55749cf07e6e4ebbacb59d5b5b25a493950660b0/.github/workflows/build.yml#L21 - but thats exactly when GA complains that the Action (which specifies https://github.com/samuelmeuli/action-electron-builder/blob/master/action.yml#L56 to use Node-12) results in The "action" should be updated to use newer versions of Node - not the Workflow |
@njmulsqb It's not about the workflow you use it in. It's about the underlying dependencies that this action has (Node 12) which is why the action itself needs to be updated. |
Exactly - So my earlier comment.
After all then. |
Yep! I forked the repo and changed it myself and everything worked fine. |
@petervanderwalt Sorry, I am pretty new to github actions so couldnt understand your statement technically. I just changed the node version in my local workflows file and this action is now working fine with me and is producing releases perfectly. |
I have forked the repo into my companies organization. It's a public action and can be found here. Just a note that the readme is out of date as the action name has obviously changed. Simply click on the "use latest version" button to get the new action name |
Figured, thanks! |
Thank you so much! |
Now you can use https://github.com/marketplace/actions/electron-builder-action-using-node-20 for node 20
|
Thanks for the update, we forked at the time and figured it out. Also adapted the action to do the stuff we wanted, so we are good for now, but glad you updated it for everyone else (: |
I get this message:
The text was updated successfully, but these errors were encountered: