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

fix: source archive not being added when no extra-files #3938

Merged
merged 3 commits into from Apr 13, 2023
Merged

Conversation

caarlos0
Copy link
Member

closes #3937

closes #3937

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 13, 2023
@caarlos0 caarlos0 self-assigned this Apr 13, 2023
@caarlos0 caarlos0 added the bug Something isn't working label Apr 13, 2023
@cloudflare-pages
Copy link

cloudflare-pages bot commented Apr 13, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4a98220
Status: ✅  Deploy successful!
Preview URL: https://2d5e5dbd.goreleaser.pages.dev
Branch Preview URL: https://fix-src.goreleaser.pages.dev

View logs

Comment on lines -95 to -98
// zips wont have the parent dir
if format == "zip" {
expected = expected[1:]
}
Copy link
Member Author

Choose a reason for hiding this comment

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

this was actually another bug, fixed it too

@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #3938 (4a98220) into main (b36e30a) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3938      +/-   ##
==========================================
+ Coverage   83.54%   83.58%   +0.03%     
==========================================
  Files         124      124              
  Lines       10954    10958       +4     
==========================================
+ Hits         9152     9159       +7     
+ Misses       1445     1443       -2     
+ Partials      357      356       -1     
Impacted Files Coverage Δ
internal/pipe/sourcearchive/source.go 76.00% <100.00%> (+3.83%) ⬆️
pkg/archive/zip/zip.go 67.39% <100.00%> (+0.35%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@caarlos0 caarlos0 merged commit f6b5e9a into main Apr 13, 2023
12 checks passed
@caarlos0 caarlos0 deleted the fix-src branch April 13, 2023 14:44
@github-actions github-actions bot added this to the v1.18.0 milestone Apr 13, 2023
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/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Source archive are not added as an artifact
1 participant