-
Notifications
You must be signed in to change notification settings - Fork 164
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
set-output is deprecated #234
Comments
It would be great if we can catch this. However, how can we check it statically? The deprecated command is run by outputting a command to stdout dynamically. |
@rhysd you mean statically check if a command is deprecated? |
actionlint itself is using the deprecated command. e.g. https://github.com/rhysd/actionlint/blob/main/scripts/download-actionlint.bash#L135 Catching usage of this deprecated command would be useful, but I don't think that's what this issue was requesting. |
No. I meant statically detecting the command is used.
Yes. I said this idea would be great if it can be realized without false positive in #234 (comment). |
Today I have some time to implement this with simple pattern matching. I will try it and look if it causes some false positives or not. |
v1.6.22 included this feature. |
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
I am getting the following warning when running actionlint
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/The text was updated successfully, but these errors were encountered: