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

Warning related to "set-output" being depricated on ci #5

Closed
NiharR027 opened this issue Jul 3, 2024 · 1 comment
Closed

Warning related to "set-output" being depricated on ci #5

NiharR027 opened this issue Jul 3, 2024 · 1 comment

Comments

@NiharR027
Copy link

NiharR027 commented Jul 3, 2024

Thank you for this package 🙏

When the job does Run thomaseizinger/[email protected]

I saw this on the ci:

Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

It printed this 4 times.

From looking at the code, I think the error is coming out from this file: https://github.com/thomaseizinger/create-release/blob/master/src/index.ts

Specifically where it does:

setOutput('id', releaseId.toString());
setOutput('url', url);
setOutput('html_url', htmlUrl);
setOutput('upload_url', uploadUrl);

Hence the ci prints 4 warnings, i.e. 1 time for each setOutput.

I tried to do some research and found an open issue about this: actions/toolkit#1218 (maybe that's related and we can find some solution from there?)

@thomaseizinger
Copy link
Owner

Fixed in #7.

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

No branches or pull requests

2 participants