Skip to content

Bump Action to Node 20#43

Open
kkaulen wants to merge 1 commit intoMarceloPrado:masterfrom
kkaulen:master
Open

Bump Action to Node 20#43
kkaulen wants to merge 1 commit intoMarceloPrado:masterfrom
kkaulen:master

Conversation

@kkaulen
Copy link
Contributor

@kkaulen kkaulen commented Jun 4, 2024

As Node 16 nears its EOL, users of this action get a message that it should be upgraded to use node 20. This PR performs this upgrade while also bumping the dependencies of this action.

As the ubuntu-latest image does not support Node 20 yet, I also added a step in the testing workflow to setup a node 20 environment.

bump checkout action in testing workflow

fix yarn lock file

use node 20 in testing workflow

Bump action to node20
@mrprkr
Copy link

mrprkr commented Aug 19, 2024

Could we get this merged? GH actions is throwing a warning 16 is deprecated.

@lundcm
Copy link

lundcm commented Aug 30, 2024

@MarceloPrado - can you take a look at this please? Thank you!

@ktamas77
Copy link

i was about to request this - upgrade looks good - @MarceloPrado could you merge this please?

@galethil
Copy link

@MarceloPrado could you please look at this and merge? If will fix the warnings in GitHub actions.

@ktamas77
Copy link

ktamas77 commented Nov 4, 2024

Since th maintainer abandoned the project, I fixed this plus added a new branch parameter to be able to run it in PRs as well: https://github.com/ktamas77/has-changed-path

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.

7 participants