Skip to content

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Apr 19, 2022

currently it does not handle universal binaries very well if there are ids different than the default

This makes the default ids filter empty, so it actually should archive all, not just the ids from the builds section.

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@caarlos0 caarlos0 self-assigned this Apr 19, 2022
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 19, 2022
@vercel vercel bot temporarily deployed to Preview April 19, 2022 01:17 Inactive
@caarlos0 caarlos0 added the bug Something isn't working label Apr 19, 2022
@codecov
Copy link

codecov bot commented Apr 19, 2022

Codecov Report

Merging #3052 (6ad8ee5) into main (38bc6b5) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3052      +/-   ##
==========================================
- Coverage   84.51%   84.50%   -0.01%     
==========================================
  Files         112      112              
  Lines        9190     9185       -5     
==========================================
- Hits         7767     7762       -5     
  Misses       1149     1149              
  Partials      274      274              
Impacted Files Coverage Δ
internal/pipe/archive/archive.go 94.37% <100.00%> (-0.12%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38bc6b5...6ad8ee5. Read the comment docs.

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@pull-request-size pull-request-size bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 20, 2022
@caarlos0 caarlos0 marked this pull request as ready for review April 20, 2022 01:34
@vercel vercel bot temporarily deployed to Preview April 20, 2022 01:34 Inactive
@caarlos0 caarlos0 merged commit 922992c into main Apr 20, 2022
@caarlos0 caarlos0 deleted the archive-ids branch April 20, 2022 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant