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

chore(ci) assign unique names to uploaded artifacts #506

Closed
wants to merge 3 commits into from

Conversation

casimiro
Copy link
Contributor

Since v4, the action upload-artifact no longer supports uploading artifacts with non-unique names.

Copy link

codecov bot commented Feb 14, 2024

Codecov Report

Merging #506 (c340263) into main (45a0c12) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##                main        #506   +/-   ##
=============================================
  Coverage   90.02022%   90.02022%           
=============================================
  Files             46          46           
  Lines           9890        9890           
=============================================
  Hits            8903        8903           
  Misses           987         987           
Flag Coverage Δ
unit 90.08398% <ø> (ø)
valgrind 77.86378% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@casimiro casimiro force-pushed the chore/fix-upload-artifact branch from c53abdb to 5153c09 Compare February 14, 2024 16:43
@casimiro casimiro added the pr/work-in-progress PR: Work in progress label Feb 14, 2024
@casimiro casimiro force-pushed the chore/fix-upload-artifact branch from 5153c09 to 4c0fa28 Compare February 14, 2024 17:33
Since v4, action `upload-artifact` no longer supports uploading artifacts with
non-unique names.
* `docker/login-action` to v3
* `docker/setup-qemu-action` to v3
@casimiro casimiro force-pushed the chore/fix-upload-artifact branch from 3c67dc3 to c340263 Compare February 14, 2024 22:56
@hishamhm hishamhm mentioned this pull request Feb 15, 2024
3 tasks
@thibaultcha
Copy link
Member

thibaultcha commented Feb 15, 2024

@casimiro Caio, would you mind drafting a PR that takes both @hishamhm's work in #486 and the bits from this PR? Instead of merging #486 and then rebasing this, rather rebasing this PR atop #486. I think name: release-artifacts-${{ github.job }} is a good template for all these artifacts names (combining both of your ideas). This PR also has the unit/valgrind uploads fixed and some other version bumps. Let's combine it all and enable the release job temporarily in release.yml to test it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/work-in-progress PR: Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants