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

Update node version and actions/core to 1.10.0 #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jceresini
Copy link

Github is deprecating the set-output command in ~3 weeks, and this action will stop working. They are also deprecating node:12. It looks like this is unmaintained, but just in case you're willing to accept a PR, this should address both issues.

Here are the warnings that we've seen when using this action recently:

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: jitterbit/get-changed-files@v1. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.
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/

And some quotes from them:

We are monitoring telemetry for the usage of these commands and plan to fully disable them on 31st May 2023. Starting 1st June 2023 workflows using save-state or set-output commands via stdout will fail with an error.
...
Action authors who are using the toolkit should update the @actions/core package to v1.10.0 or greater to get the updated saveState and setOutput functions.

This PR just updates versions in package.json and the actions.yaml. Then I ran yarn package to rebuild dist/index.js

@a-chatterjee
Copy link

I need this change too.

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