Skip to content

🩹 fix deprecation warning: Node.js 12 actions are deprecated. - #71

Merged
yubiuser merged 1 commit into
yubiuser:developmentfrom
thomasmerz:fix_node12_deprecation
Dec 14, 2022
Merged

🩹 fix deprecation warning: Node.js 12 actions are deprecated.#71
yubiuser merged 1 commit into
yubiuser:developmentfrom
thomasmerz:fix_node12_deprecation

Conversation

@thomasmerz

Copy link
Copy Markdown
Contributor

For more information see:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

Please update the following actions to use Node.js 16: actions/checkout@master
Using version v3, latest or master: actions/checkout#689

@thomasmerz
thomasmerz changed the base branch from master to development December 14, 2022 14:54
@thomasmerz

Copy link
Copy Markdown
Contributor Author

See actions before with this deprecation warning.

@yubiuser

Copy link
Copy Markdown
Owner

Thanks for your PR. I suspect there is a bug in dependabot which did not recognize version 2 in this workflow (it only checked version 3 for updates - which the other two workflows used).

@yubiuser yubiuser left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to set explicit version numbers to get changelogs and updates via dependabot.

Comment thread .github/workflows/shellcheck.yml Outdated
   Node.js 12 actions are deprecated.
   For more information see:
   https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.
   Please update the following actions to use Node.js 16: actions/checkout@master

   Using version v3, latest or master: actions/checkout#689
@yubiuser
yubiuser merged commit c8d111e into yubiuser:development Dec 14, 2022
@yubiuser yubiuser mentioned this pull request Dec 14, 2022
@thomasmerz
thomasmerz deleted the fix_node12_deprecation branch December 16, 2022 19:04
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