-
Notifications
You must be signed in to change notification settings - Fork 79
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
Test for act #111
Test for act #111
Conversation
mhm, this will also fail since GHA runners don't have enough space for that image ( |
What's an image that would fit?
…On Wed, Nov 17, 2021, 15:32 Ryan ***@***.***> wrote:
mhm, this will also fail since GHA runners don't have enough space for
that image (full-20.04 is +40GB)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#111 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA63A35MNECY2ZHJHKPAXTUMQNPLANCNFSM5IBZBDAA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
well, there you go, (act) works :) |
thanks a lot @catthehacker |
81f1eda
to
43ace0c
Compare
Co-authored-by: Ryan <me@hackerc.at>
Co-authored-by: Ryan <me@hackerc.at>
Co-authored-by: Ryan <me@hackerc.at>
Co-authored-by: Ryan <me@hackerc.at>
Co-authored-by: Ryan <me@hackerc.at>
@catthehacker does act respect GITHUB_ENV? It doesn't seem to be setting $PATH |
It does and
|
|
oh well, that 2nd |
@domenkozar should be working fine now |
- uses: actions/setup-go@v2.1.4 | ||
with: | ||
go-version: '^1.16' | ||
- run: go install github.com/nektos/act@master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use @latest here to not get caught off guard by bugs in master. needed for the bug fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@catthehacker a bugfix release would be amazing :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's not up to me to decide, I only review and fix stuff
I have few small fixes that I would really like to push into next release and then I'll ping cplee to get new release
No description provided.