Skip to content

chore: upgrade actions to a version that use node20 (LTS) #1123

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

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

r4mmer
Copy link
Member

@r4mmer r4mmer commented Aug 21, 2024

Motivation

Some workflows were giving warnings that they were using actions running on node16 and node16 has reached EOL for almost a year.
For now all node16 runners are forced to run on node20 and this behavior will be set to fail the action in a few months.

Github announcement

image

Acceptance Criteria

  • upgrade actions to a version that use node20 (LTS)

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@r4mmer r4mmer self-assigned this Aug 21, 2024
Copy link

github-actions bot commented Aug 21, 2024

🐰Bencher

ReportWed, August 21, 2024 at 19:13:12 UTC
Projecthathor-core
Branchchore/upgrade-lts-node-actions
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyLatency Results
nanoseconds (ns) | (Δ%)
Latency Lower Boundary
nanoseconds (ns) | (%)
Latency Upper Boundary
nanoseconds (ns) | (%)
sync-v2 (up to 20000 blocks)✅ (view plot)101,384,972,728.80 (-1.46%)82,313,370,976.82 (81.19%)123,470,056,465.23 (82.11%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.87%. Comparing base (72ab590) to head (0796676).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1123      +/-   ##
==========================================
- Coverage   84.88%   84.87%   -0.01%     
==========================================
  Files         315      315              
  Lines       24071    24071              
  Branches     3647     3647              
==========================================
- Hits        20432    20431       -1     
  Misses       2927     2927              
- Partials      712      713       +1     

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

@r4mmer r4mmer force-pushed the chore/upgrade-lts-node-actions branch from 4e80356 to 4904543 Compare August 21, 2024 18:16
@r4mmer r4mmer force-pushed the chore/upgrade-lts-node-actions branch from 4904543 to 0796676 Compare August 21, 2024 19:10
@r4mmer r4mmer merged commit e85e4be into master Aug 22, 2024
13 checks passed
@r4mmer r4mmer deleted the chore/upgrade-lts-node-actions branch August 22, 2024 01:50
@jansegre jansegre mentioned this pull request Oct 4, 2024
2 tasks
@jansegre jansegre mentioned this pull request Dec 11, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants