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 actions to latest version #33

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

darthmaim
Copy link
Contributor

@darthmaim darthmaim commented Nov 22, 2024

This workflow still used actions/cache@v2 which will stop working in February. actions/cache is also no longer required, because actions/setup-node handles caching dependencies.

The other used actions are also still using node 12 and were automatically running as node 20, which might break them in weird ways that are difficult to detect.

Upgrading codecov/codecov-action from v2 to v5 introduced a few major changes, one of which is that v4 now always requires a token, but v5 relaxed that again. I'm not sure if you have a token setup already, or if you need to change anything in your codecov settings. The Migration Guide is here: https://github.com/codecov/codecov-action/tree/v5?tab=readme-ov-file#v5-release

This workflow still used `actions/cache@v2` which will [stop working in February](https://github.blog/changelog/2024-09-16-notice-of-upcoming-deprecations-and-changes-in-github-actions-services/). `actions/cache` is also no longer required, because `actions/setup-node` handles caching dependencies.

The other used actions are also still using node 12 and were automatically running as node 20, which might break them in weird ways that are difficult to detect.

Upgrading `codecov/codecov-action` from `v2` to `v5` introduced a few major changes, one of which is that `v4` now always requires a token, but `v5` relaxed that again. I'm not sure if you have a token setup already, or if you need to change anything in your codecov settings. The Migration Guide is here: https://github.com/codecov/codecov-action/tree/v5?tab=readme-ov-file#v5-release
@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.57%. Comparing base (52beccd) to head (d180b81).
Report is 23 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##            master      #33      +/-   ##
===========================================
- Coverage   100.00%   98.57%   -1.43%     
===========================================
  Files           14       15       +1     
  Lines          164      210      +46     
  Branches        51       68      +17     
===========================================
+ Hits           164      207      +43     
- Misses           0        3       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@queicherius queicherius merged commit 2a95eb8 into gw2efficiency:master Nov 23, 2024
2 checks passed
@queicherius
Copy link
Member

Thanks! TIL you dont need the extra cache action anymore.

@darthmaim darthmaim deleted the feature/update-ci branch November 23, 2024 03:23
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.

3 participants