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: Update deprecated set-output format for GitHub Actions to use environment variables #61

Merged
merged 2 commits into from
Jun 21, 2023

Conversation

sevenseacat
Copy link
Contributor

See https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/ for more details

Closes #60

(Note: I have merely proved it correct, not tried it)

Contributor checklist

  • My commit messages follow the Conventional Commit Message Format
    For example: fix: Multiply by appropriate coefficient, or
    feat(Calculator): Correctly preserve history
    Any explanation or long form information in your commit message should be
    in a separate paragraph, separated by a blank line from the primary message
  • Bug fixes include regression tests
  • Features include unit/acceptance tests

@sevenseacat
Copy link
Contributor Author

Actually... I think I need to try it because I'm not sure it will work right now

@sevenseacat sevenseacat marked this pull request as draft June 15, 2023 07:29
@sevenseacat
Copy link
Contributor Author

So I think this is right now. According to the blog, we can write to the file in GITHUB_OUTPUT, with the format {key}={value} so that's what I've done. It works in my testing!

Screenshot 2023-06-21 at 1 50 33 pm

@sevenseacat sevenseacat marked this pull request as ready for review June 21, 2023 05:51
@zachdaniel zachdaniel merged commit b1ea051 into zachdaniel:master Jun 21, 2023
@sevenseacat sevenseacat deleted the fix/set-output-deprecation branch July 20, 2023 01:36
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.

Deprecation warnings raised when using the GitHub Actions formatter
2 participants