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.

Sign up for GitHub

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

Jump to bottom

Update deprecated GitHub actions #1913

Merged
merged 2 commits into from Sep 8, 2023
Merged

Update deprecated GitHub actions #1913

merged 2 commits into from Sep 8, 2023

Conversation

AdeelH
Copy link
Collaborator

@AdeelH AdeelH commented Sep 7, 2023 •
edited

Overview

This PR addresses the following warnings from GitHub Actions:

image

image

Checklist

  • Added needs-backport label if PR is bug fix that applies to previous minor release
  • Ran scripts/format_code and committed any changes
  • Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

Notes

N/A

Testing Instructions

  • Check if warnings go away.

@AdeelH AdeelH added the needs-backport This PR needs to be backported to release branches label Sep 7, 2023
@AdeelH AdeelH changed the title actions/checkout@v2 --> actions/checkout@v3 Update deprecated GitHub actions Sep 8, 2023
@AdeelH
actions/checkout@v2 --> actions/checkout@v3
008a4b7 
Addresses the following warning from GitHub Actions:
"The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, codecov/codecov-action@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/"
@AdeelH
codecov/codecov-action@v2 --> codecov/codecov-action@v3
50dd064
@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #1913 (815a8bc) into master (3acbdd0) will decrease coverage by 0.02%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head 815a8bc differs from pull request most recent head 50dd064. Consider uploading reports for the commit 50dd064 to get more accurate results

@@            Coverage Diff             @@
##           master    #1913      +/-   ##
==========================================
- Coverage   81.57%   81.56%   -0.02%     
==========================================
  Files         190      190              
  Lines        9282     9282              
==========================================
- Hits         7572     7571       -1     
- Misses       1710     1711       +1

see 1 file with indirect coverage changes

📢 Have feedback on the report? Share it here.

@AdeelH AdeelH marked this pull request as ready for review September 8, 2023 15:14
@AdeelH AdeelH merged commit 32eb656 into azavea:master Sep 8, 2023
2 checks passed
@AdeelH AdeelH deleted the actions_checkout branch September 8, 2023 15:17
@AdeelH AdeelH mentioned this pull request Sep 22, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
needs-backport This PR needs to be backported to release branches
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@AdeelH