Skip to content

Update action.yml#56

Merged
kt3k merged 1 commit intodenoland:mainfrom
ctdeakin:patch-1
Jan 28, 2024
Merged

Update action.yml#56
kt3k merged 1 commit intodenoland:mainfrom
ctdeakin:patch-1

Conversation

@ctdeakin
Copy link
Copy Markdown
Contributor

Update action.yml to use node20 because node16 is deprecated

Update action.yml to use node20 because node16 is deprecated
@uncenter
Copy link
Copy Markdown

Thanks, I was about to do this. You could probably update the PR title to be more descriptive of the changes fyi.

Copy link
Copy Markdown
Contributor

@kt3k kt3k left a comment

Choose a reason for hiding this comment

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

LGTM

@bjesuiter
Copy link
Copy Markdown

bjesuiter commented Jan 28, 2024

Can this be merged? Would be nice to get rid of the warning! :)

Fixes #55

@kt3k
Copy link
Copy Markdown
Contributor

kt3k commented Jan 28, 2024

Thanks for the contributions and feedbacks! We'll release soon after basic testing of the new version

@kt3k kt3k merged commit ef28d46 into denoland:main Jan 28, 2024
@kt3k kt3k mentioned this pull request Jan 28, 2024
@LitoMore LitoMore mentioned this pull request Jan 28, 2024
marvinhagemeister pushed a commit to denoland/fresh that referenced this pull request Jan 29, 2024
I tested the change in my own repo, the warning on github disapears for
the "actions/checkout" action.
It is still there for the denoland/setup-deno action, but an upgrade PR
is already ongoing for this action, see:
denoland/setup-deno#56
@ctdeakin ctdeakin deleted the patch-1 branch January 30, 2024 23:10
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.

4 participants