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

GitHub Artifacts API update #12210

Merged
merged 22 commits into from
Feb 3, 2025
Merged

GitHub Artifacts API update #12210

merged 22 commits into from
Feb 3, 2025
+264 −279

Conversation

4e6
Copy link
Contributor

@4e6 4e6 commented Jan 31, 2025 •
edited
Loading

Pull Request Description

close #9976

Changelog:

  • update: remove calls to the outdated uploads API in the run script and replace them with actions/upload-artifact@v4 and actions/download-artifact@v4 actions

Important Notes

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • All code follows the
    Scala,
    Java,
    TypeScript,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • Unit tests have been written where possible.

@4e6 4e6 added the CI: No changelog needed Do not require a changelog entry for this PR. label Jan 31, 2025
@4e6 4e6 self-assigned this Jan 31, 2025
@4e6 4e6 marked this pull request as ready for review February 2, 2025 20:15
@jdunkerley jdunkerley merged commit 9daf16d into develop Feb 3, 2025
42 of 46 checks passed
@jdunkerley jdunkerley deleted the wip/db/9976-github-artifacts-api-update branch February 3, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdunkerley jdunkerley jdunkerley approved these changes

@farmaazon farmaazon farmaazon approved these changes

@AdRiley AdRiley Awaiting requested review from AdRiley AdRiley is a code owner

@hubertp hubertp Awaiting requested review from hubertp hubertp is a code owner

@Frizi Frizi Awaiting requested review from Frizi Frizi is a code owner

Assignees

@4e6 4e6

Labels
CI: No changelog needed Do not require a changelog entry for this PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GitHub Artifacts API update
3 participants
@4e6 @farmaazon @jdunkerley