Skip to content

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Apr 20, 2022

if the user provide custom targets without the goamd64, goarm or gomips bit, things may go awry at some point.

this prevents issues by suffixing them with the default when its missing.

closes #3055

if the user provide custom targets without the goamd64, goarm or gomips bit, things may go awry at some point.

this prevents issues by suffixing them with the default when its missing.

closes #3055

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@caarlos0 caarlos0 self-assigned this Apr 20, 2022
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 20, 2022
@caarlos0 caarlos0 added the bug Something isn't working label Apr 20, 2022
@caarlos0 caarlos0 marked this pull request as ready for review April 20, 2022 01:47
@vercel vercel bot temporarily deployed to Preview April 20, 2022 01:48 Inactive
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@pull-request-size pull-request-size bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 20, 2022
@vercel vercel bot temporarily deployed to Preview April 20, 2022 02:34 Inactive
@codecov
Copy link

codecov bot commented Apr 20, 2022

Codecov Report

Merging #3056 (40911a2) into main (38bc6b5) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3056      +/-   ##
==========================================
+ Coverage   84.51%   84.53%   +0.01%     
==========================================
  Files         112      112              
  Lines        9190     9199       +9     
==========================================
+ Hits         7767     7776       +9     
  Misses       1149     1149              
  Partials      274      274              
Impacted Files Coverage Δ
internal/builders/golang/build.go 91.17% <100.00%> (+0.55%) ⬆️
internal/pipe/archive/archive.go 94.37% <0.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...40911a2. Read the comment docs.

@caarlos0 caarlos0 merged commit 6343639 into main Apr 20, 2022
@caarlos0 caarlos0 deleted the custom-targets branch April 20, 2022 12:43
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.

scoop publishing stopped working when goarm64 support was added
1 participant