Skip to content

updating to actions/checkout@v3 in order to avoid the set-output depr…#164

Merged
dorny merged 1 commit intodorny:masterfrom
IdanYaffe:idanyaffe-master
Oct 12, 2022
Merged

updating to actions/checkout@v3 in order to avoid the set-output depr…#164
dorny merged 1 commit intodorny:masterfrom
IdanYaffe:idanyaffe-master

Conversation

@IdanYaffe
Copy link
Copy Markdown
Contributor

…ecation warning

@dorny dorny merged commit 027a82c into dorny:master Oct 12, 2022
@IdanYaffe
Copy link
Copy Markdown
Contributor Author

@dorny I'm afraid that this is not enough.
looks like there's also a need in updating the "setOutput" call as per this doc:
https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions

here are the places that I could find that are in need of this change:
https://github.com/dorny/paths-filter/search?q=%22setOutput%22

@IdanYaffe
Copy link
Copy Markdown
Contributor Author

@dorny looks like it was addressed by this PR in the "actions/typescript-action" repo:
https://github.com/actions/typescript-action/pull/611/files#diff-468016aae139513ca1e5421e59385204d06350f7fb00672a31184f70f4eb3425

@dorny
Copy link
Copy Markdown
Owner

dorny commented Oct 12, 2022

@IdanYaffe thanks for the notice. Here's a blog post about it - https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

So updating the package should be enough. I will do it now.

@IdanYaffe
Copy link
Copy Markdown
Contributor Author

IdanYaffe commented Oct 12, 2022

@dorny I'm afraid that's still not resolved.
if I'm not mistaken, we need to update the setOutput function in the index.js:
https://raw.githubusercontent.com/dorny/paths-filter/master/dist/index.js

should be something like this:
https://github.com/actions/typescript-action/blob/main/dist/index.js#L355-L362

@dorny
Copy link
Copy Markdown
Owner

dorny commented Oct 12, 2022

Yes, I have to update the index.js. It's a packaged version of all the code of this action and its dependencies. I usually do it just once after multiple PRs are merged, together with adding a new tag. The file is big so every change of it increases the size of the repo. Looks like I won't make more changes today so I will do a release shortly.

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