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

[BUG]: minimum version of node (node 18) is higher than available in github actions (node 16) #2505

Closed
1 task done
AWare opened this issue Jul 24, 2023 · 13 comments
Closed
1 task done
Labels
Type: Bug Something isn't working as documented Type: Support Any questions, information, or general needs around the SDK or GitHub APIs
Projects

Comments

@AWare
Copy link

AWare commented Jul 24, 2023

What happened?

Currently the maximum version of node supported in a github action is 16.

actions/runner#2704

Using this gives an error as per #2497

It's very frustrating that these two major github products have incompatible node versions.

Please can support for node 16 be retained until a later version is made available on gha?

Versions

n/a

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@AWare AWare added Status: Triage This is being looked at and prioritized Type: Bug Something isn't working as documented labels Jul 24, 2023
@ghost ghost added this to Bugs in JS Jul 24, 2023
@AWare AWare changed the title [BUG]: minimum version of node is higher than available in github actions [BUG]: minimum version of node (node 18) is higher than available in github actions (node 16) Jul 24, 2023
@wolfy1339
Copy link
Member

This was an intentional change. We have the @octokit/action package that currently supports NodeJS 16 for the time being that is intended to be used on GitHub actions

@wolfy1339
Copy link
Member

If you can, you can simply run your JS code using the @actions/setup-node action to use Node v18

@kfcampbell kfcampbell removed the Status: Triage This is being looked at and prioritized label Jul 24, 2023
@AWare
Copy link
Author

AWare commented Jul 25, 2023

while a good workaround is linked in the discussion (actions/runner#2704 (comment)) we use actions extensively and this represents a not insignificant refactor.

while this was an intentional change, it is not reasonable for github to have different nodejs supported versions for two major products that would foreseeably be used together

@wolfy1339 thank you for the response though, I appreciate your work as a volunteer. and my displeasure is entirely at github product management for allowing this incompatibility to occur.

@nickfloyd nickfloyd added the Type: Support Any questions, information, or general needs around the SDK or GitHub APIs label Jul 31, 2023
@VictorChacon-Ada
Copy link

Hello, I just tried to use the @octokit/action package to workaround this issue, but even in that I get a compatibility issue on @octokit/plugin-paginate-rest@8.0.0.

@nickfloyd
Copy link
Contributor

Hey @VictorChacon-Ada so sorry for the trouble you're experiencing. Would you be able to share a repo, or actions yaml, or more details so that we can get a better idea of what is going on? Thanks. I hope to hear from you soon!

@VictorChacon-Ada
Copy link

I don't have a specific repo example, I was working on a brand new action.
I'm running node v16.16.0. If I start a brand new project with yarn init and then yarn add @octokit/action, I get the error

@octokit/plugin-rest-endpoint-methods@9.0.0: The engine "node" is incompatible with this module. Expected version ">= 18". Got "16.16.0"

If I manually set the version to 4.x, the package is succesfully installed, but I expected this specific package to still be compatible with the only current node version available in github actions.

@nickfloyd
Copy link
Contributor

Hey @VictorChacon-Ada

Thanks for getting back to us. Thanks for the explanation. We'll have a look at things using the details you provided to see if we can get a better picture of what might be going on. We're a little short handed this week but I'll see if I can get something worked out and reach back out next week at the latest.

Again, apologies for the trouble here.

@nickfloyd
Copy link
Contributor

while this was an intentional change, it is not reasonable for github to have different nodejs supported versions for two major products that would foreseeably be used together

Hey @AWare,

Thank you for the candor and perspective here. I'm sorry that the way this change was done has made things problematic for you. While there are work arounds, you are correct, this could've gone a bit smoother. For Octokit.js we had to make a decision to move forward given we were using an EoL version such as v14.

We are really sorry for the trouble and want you to know that we are doing our best to make processes better by having actionable discussions like this and this to help everyone navigate things like coordinated dependencies and breaking changes - both internally and externally.

We take your comments to ❤️ and are grateful that you are part of this community making it better.

@nickfloyd
Copy link
Contributor

@VictorChacon-Ada I haven't had a chance to have a look at this yet, but now that @kfcampbell is back this week we should be able to take a look shortly.

@kfcampbell
Copy link
Member

@VictorChacon-Ada what version of Yarn and Node are you using? I'm unable to reproduce that case locally using Yarn v1.22.19 and Node v18.14.2. If you're seeing that error on GitHub Actions runners, can you use actions/setup-node to install Node v18 beforehand?

@VictorChacon-Ada
Copy link

@kfcampbell the problem I had was running Node v16.6.0, with yarn v1.22.15.
For now, I have used the workaround of wrapping my action as a composite action, and using setup-node for node 18 before. But by the previous messages in this thread, I understood that the package @octokit/action should still work with node 16.

@AWare
Copy link
Author

AWare commented Aug 24, 2023

node 20 is now available as a target, so I think this issue can probably be closed

@wolfy1339
Copy link
Member

Closing as per the latest comment. Node JS 20 is now avaiable as a target on GitHub Actions Runners

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented Type: Support Any questions, information, or general needs around the SDK or GitHub APIs
Projects
Archived in project
JS
  
Bugs
Development

No branches or pull requests

5 participants