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

feat: apk support #207

Merged
merged 72 commits into from
Aug 17, 2020
Merged

feat: apk support #207

merged 72 commits into from
Aug 17, 2020

Conversation

caarlos0
Copy link
Member

closes #126


its simpler than the original PR (no signature support).

@vercel vercel bot temporarily deployed to Preview August 16, 2020 17:44 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 16, 2020 17:47 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 16, 2020 18:06 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 16, 2020 18:23 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 17, 2020 13:49 Inactive
@vercel vercel bot temporarily deployed to Preview August 17, 2020 17:13 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 17, 2020 17:32 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 17, 2020 17:36 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 17, 2020 17:46 Inactive
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 17, 2020 18:02 Inactive
@codecov
Copy link

codecov bot commented Aug 17, 2020

Codecov Report

Merging #207 into master will increase coverage by 0.48%.
The diff coverage is 71.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   69.76%   70.25%   +0.48%     
==========================================
  Files           6        7       +1     
  Lines         688      911     +223     
==========================================
+ Hits          480      640     +160     
- Misses        131      164      +33     
- Partials       77      107      +30     
Impacted Files Coverage Δ
cmd/nfpm/main.go 0.00% <ø> (ø)
apk/apk.go 71.74% <71.74%> (ø)

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 62c61a5...c23e600. Read the comment docs.

Copy link
Contributor

@djgilcrease djgilcrease left a comment

Choose a reason for hiding this comment

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

LGTM

@bhamail
Copy link
Contributor

bhamail commented Aug 17, 2020

LGTM! Thanks @caarlos0 for picking this up!

BTW, I'm totally fine with the need for --allow-untrusted for now, especially since apk isn't the only one unsigned.

@caarlos0 caarlos0 merged commit 2e97c8d into master Aug 17, 2020
@caarlos0 caarlos0 deleted the apk_play branch August 17, 2020 20:28
@caarlos0
Copy link
Member Author

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@goreleaser goreleaser locked as resolved and limited conversation to collaborators Nov 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants