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

Upgrade to the latest version of the setup-go github action #1040

Merged
merged 1 commit into from
Nov 25, 2020

Conversation

AntonNguyen
Copy link
Contributor

@AntonNguyen AntonNguyen commented Nov 25, 2020

The github action pipeline is currently broken, giving us the following error:

Run actions/setup-go@v2-beta
Setup go  version spec 1.14
Error: Unable to process command '::set-env name=GOROOT::/opt/hostedtoolcache/go/1.14.12/x64' successfully.
Error: The `set-env` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

The beta version utilizes the deprecated set-env and add-path commands. In the 2.1.3 release, they've introduced a fix (actions/setup-go#76) by upgrading the core library. More details can be found at https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

Sample faling build: https://github.com/go-kit/kit/pull/1013/checks?check_run_id=1453805584

The beta version utilizes the deprecated `set-env` and `add-path` commands
@peterbourgon
Copy link
Member

Agh!! Thank you!!

@peterbourgon peterbourgon merged commit 15dfca1 into go-kit:master Nov 25, 2020
@AntonNguyen AntonNguyen deleted the fix-gha branch November 25, 2020 19:32
@sagikazarmark sagikazarmark added this to the v0.11.0 milestone Jun 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants