-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Enhancement: node16 runner #910
Comments
|
Then we can wait until it is 😄 |
I've already added TODO tasks for
|
Hi, I'm getting that error:
I'm running act with:
I have created this sample repo to reproduce the error. Is this already working @catthehacker? I see there are two pending tasks above but the PR was merged. |
It is, but most likely you are using v0.2.25 which doesn't have that change |
👋 @catthehacker any thought on when:
Is going to be completed. I am using the medium image and node16 but getting this error, it would be great if this was supported 💯 ❤️ I ensured I was using the latest and greatest |
Installing the latest master build with |
Really 🤔 This is how I am installing it. E.G
I re-build my environment every night to pull the latest and greatest. I don't have I just double-checked and it doesn't work for me. 🤔 (Something Codespaces is meant to solve haha) You are welcome to pull my codespace and test it out, but it isn't working for me 👀 |
From what I can tell from reading the install script it downloads binaries from the releases, but the changes have not been released yet, hence having to install from master For me personally installing go just to build tools from source code has been worth it instead of downloading binaries and setting that up, despite not actually developing anything in go. But if you're using codespaces I can see why that would be less desirable |
100%! If I was on my local machine I would so install Go and get this done, but it's more overhead on my system having to install Go when it actually isn't needed. Local machine 100% I would install from Go, but as no other Go is indeed I would rather not if possible ❤️
Ah okay this explains it 😄 Do we know when the changes may get released? 👀 |
You can get latest build from https://github.com/nektos/act/actions/runs/1773288141 |
Hey @catthehacker, correct, but that doesn't work with this script. https://github.com/nektos/act/blob/master/install.sh It would be great if it could be generally available :) |
@catthehacker This is the latest version available through homebrew and is listed as release in the repository. |
That is correct, though I don't understand what's your point? |
You wrote that in version 0.2.25 there is no node16 support yet. What version is it in? Act latest release - 0.2.25 : https://github.com/nektos/act/releases/latest Here is the brew output:
|
None, there is no version released yet with that change |
And when is the release planned? |
use |
|
Describe feature
The
node16
has finally been addedhttps://github.com/actions/runner/releases/tag/v2.285.0
actions/runner#772
actions/runner#1439
Currently I get this error when trying to run (after changing the image used to
node:16-buster-slim
):Error: The runs.using key in action.yml must be one of: [composite docker node12], got node16
The text was updated successfully, but these errors were encountered: