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 action.yml #56

Merged
merged 1 commit into from
Jan 28, 2024
Merged

Update action.yml #56

merged 1 commit into from
Jan 28, 2024

Conversation

ctdeakin
Copy link
Contributor

Update action.yml to use node20 because node16 is deprecated

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

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

Copy link
Member

@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

bjesuiter commented Jan 28, 2024

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

Fixes #55

@kt3k
Copy link
Member

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
15 checks passed
@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.

None yet

4 participants