Green-Software-Foundation / carbon-aware-sdk

Carbon-Aware SDK
https://carbon-aware-sdk.greensoftware.foundation/
MIT License
484 stars 99 forks source link

Upgrade actions/{upload,download}-artifact #565

Closed YaSuenag closed 1 month ago

YaSuenag commented 2 months ago

Pull Request

Issue Number: #564

Summary

Upgrade actions/upload-artifact and actions/download-artifact in GHA workflow.

See GitHub Blog for details: https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

Changes

All of lines in 2.a-deploy.yaml.ignore are commented out, but I upgraded because this workflow might be enabled again in future.

One of actions/download-artifact in 1-pr.yaml uses v3 (non-deprecated version), but I upgraded because I think it is better to use same version entire workflows.

Checklist

Are there API Changes?

No

Is this a breaking change?

No

This PR Closes #564

YaSuenag commented 2 months ago

CC: @danuw

danuw commented 2 months ago

@YaSuenag sorry missed that. I did not update the workflow that is disabled but I guess we can close this and use the other one I fixed in the branch directly. Did not get a chance to apply the change until now but looks all good now Thank you for that btw

YaSuenag commented 2 months ago

Changes for .github/workflows/1-pr.yaml does not seems to happen in #562 - I think we will see same error when other PR is filed.

@danuw Should I revise this PR for 1-pr.yaml?

danuw commented 1 month ago

Already fixed in Dev, so closing ...