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

IP-244 - Update actions/upload-artifacts to v4 due to v3 deprecation. #25

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

owenlittlejohns
Copy link
Member

@owenlittlejohns owenlittlejohns commented Dec 11, 2024

Description

(Same as for earthdata-varinfo)

I received a deprecation notice for one of the actions we use to upload artefacts during our CI/CD runs (coverage reports and JUnit style test results output).

For more information see here.

I don't think there are any breaking changes as we only use very basic options for the action and use unique names for each artefact we upload.

Jira Issue ID

N/A

Local Test Steps

N/A - just check there were artefacts saved to the workflow run for this PR. (See here - I'm only seeing one artefact. But I think that's something to do with the Swath Projector specifically. It has some weird logging etc for tests that we should probably rip out at some point to simplify things.)

PR Acceptance Checklist

All are non-applicable. These changes are internal to our CI/CD workflows and will not require a release of any service code.

  • Jira ticket acceptance criteria met.
  • CHANGELOG.md updated to include high level summary of PR changes.
  • docker/service_version.txt updated if publishing a release.
  • Tests added/updated and passing.
  • Documentation updated (if needed).

Copy link
Member

@flamingbear flamingbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@owenlittlejohns owenlittlejohns merged commit 6253559 into main Dec 11, 2024
4 checks passed
@owenlittlejohns owenlittlejohns deleted the IP-244-upload-artifacts-v4 branch December 11, 2024 20:43
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.

2 participants