Skip to content

svcauth, articles: read file owners through a portable helper - #72

Merged
andreimarinescu merged 1 commit into
mainfrom
p2/golive-portable-owner
Oct 1, 2026
Merged

andreimarinescu merged 1 commit into
mainfrom
p2/golive-portable-owner

Conversation

@andreimarinescu

Copy link
Copy Markdown
Collaborator

The release matrix includes windows/amd64, where syscall.Stat_t does not exist, so the v0.4.0 release build failed. internal/fileowner reads the uid on unix and reports none elsewhere, which keeps both owner checks failing closed off unix.

@codecov

codecov Bot commented Oct 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/fileowner/owner_unix.go 0.00% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

The release matrix includes windows/amd64, where syscall.Stat_t does not
exist, so the v0.4.0 release build failed. internal/fileowner reads the uid on
unix and reports none elsewhere, which keeps both owner checks failing closed
off unix.
@andreimarinescu
andreimarinescu force-pushed the p2/golive-portable-owner branch from 066efe9 to dc95487 Compare October 1, 2026 13:43
@andreimarinescu
andreimarinescu merged commit 9cfb540 into main Oct 1, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant