search

goreleaser / nfpm

nFPM is Not FPM - a simple deb, rpm, apk, ipk, and arch linux packager written in Go
https://nfpm.goreleaser.com/
MIT License
2.13k stars 157 forks source link

fix: properly handle files owned by fs #760

Closed caarlos0 closed 7 months ago

caarlos0 commented 8 months ago

closes https://github.com/goreleaser/nfpm/issues/738

I hardcoded all the directories owned by the filesystem package, I think it should be enough for now... we can make this configurable in the future if needed.

cloudflare-pages[bot] commented 8 months ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1cfaeaa
Status: ✅  Deploy successful!
Preview URL: https://80720181.nfpm.pages.dev
Branch Preview URL: https://fs.nfpm.pages.dev

View logs

codecov[bot] commented 8 months ago

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (c2fe69e) 75.20% compared to head (1cfaeaa) 75.15%. Report is 1 commits behind head on main.

Files Patch % Lines
files/fs.go 57.14% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #760 +/- ## ========================================== - Coverage 75.20% 75.15% -0.06% ========================================== Files 12 13 +1 Lines 2468 2475 +7 ========================================== + Hits 1856 1860 +4 - Misses 433 435 +2 - Partials 179 180 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.