-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 deprecated set-output commands #6152
Fix deprecated set-output commands #6152
Conversation
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.
Thanks for your contribution @kevinneville! I've left some suggested changes for you.
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.
Just another double quote change, otherwise looks good!
Co-authored-by: Stu Kilgore <[email protected]>
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.
@kevinneville this LGTM and thank you so much for taking the time to fix this in multiple repos. Could you do us one last favor and update your branch to reflect the latest main
changes please? CI is failing from a change made to a dependency that we have fixed in that branch already
Resolved by #6635 |
resolves #6153
Description
As communicated by Github in https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/ these commands should be replaced.
Checklist
changie new
to create a changelog entry