Skip to content

Conversation

@yubiuser
Copy link
Contributor

@yubiuser yubiuser commented Nov 22, 2024

I noticed the CI workflow uses coverallsapp/github-action@master but this caused warnings of outdated nodeversion
2024-11-22_21-41

Checking the repo, I found this thread: coverallsapp/github-action#201 which described that they switched from master to main a year ago (coverallsapp/github-action#201 (comment)).

This PR updated the workflow file to use a tagged version of coverallsapp instead of the deprecated master branch

@kurkle kurkle merged commit c53861b into chartjs:master Nov 22, 2024
12 checks passed
@yubiuser yubiuser deleted the update/dependencies branch November 23, 2024 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants