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

fix: workflow warnings #1316

Merged

Conversation

kevinneville
Copy link
Contributor

@kevinneville kevinneville commented Oct 31, 2022

Hopefully fixes the warnings we see in the builds, for instance in https://github.com/Snowflake-Labs/terraform-provider-snowflake/actions/runs/3361622276.
There were some breaking changes as documented in https://github.com/actions/github-script#breaking-changes, that I hope I fixed correctly.

I added another PR #1317 to use dependabot to create PRs going forward, this should be merged after this PR.

@kevinneville kevinneville changed the title Fix workflow warnings fix: workflow warnings Nov 1, 2022
@sfc-gh-swinkler
Copy link
Collaborator

@kevinneville can you please run go fmt ./... to format your code?

@sfc-gh-swinkler sfc-gh-swinkler merged commit 6f513c2 into Snowflake-Labs:main Nov 7, 2022
@sfc-gh-swinkler
Copy link
Collaborator

@kevinneville thank you for your contribution

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.

2 participants