Skip to content

chore: Enable protogetter linter and run fixes #2638

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.

Sign up for GitHub

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

Open
tstirrat15 wants to merge 1 commit into
base: main
Choose a base branch
from

chore: Enable protogetter linter and run fixes

767f8fa
Select commit
Loading
Failed to load commit list.
Open

chore: Enable protogetter linter and run fixes #2638

chore: Enable protogetter linter and run fixes
767f8fa
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Apr 6, 2026 in 1s

89.63% of diff hit (target 76.99%)

View this Pull Request on Codecov

89.63% of diff hit (target 76.99%)

Annotations

Check warning on line 367 in internal/datastore/memdb/readwrite.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/memdb/readwrite.go#L367 

Added line #L367 was not covered by tests

Check warning on line 230 in internal/datastore/proxy/schemacaching/watchingcache.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/proxy/schemacaching/watchingcache.go#L230 

Added line #L230 was not covered by tests

Check warning on line 389 in internal/datastore/spanner/readwrite.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/readwrite.go#L389 

Added line #L389 was not covered by tests

Check warning on line 61 in internal/dispatch/keys/computed.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/dispatch/keys/computed.go#L61 

Added line #L61 was not covered by tests

Check warning on line 459 in internal/dispatch/remote/cluster.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/dispatch/remote/cluster.go#L459 

Added line #L459 was not covered by tests

Check warning on line 601 in internal/graph/check.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/check.go#L601 

Added line #L601 was not covered by tests

Check warning on line 1333 in internal/graph/check.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/check.go#L1333 

Added line #L1333 was not covered by tests

Check warning on line 204 in internal/graph/expand.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/expand.go#L204 

Added line #L204 was not covered by tests

Check warning on line 607 in internal/graph/lookupresources2.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lookupresources2.go#L606-L607 

Added lines #L606 - L607 were not covered by tests

Check warning on line 612 in internal/graph/lookupresources2.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lookupresources2.go#L611-L612 

Added lines #L611 - L612 were not covered by tests

Check warning on line 622 in internal/graph/lookupresources2.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lookupresources2.go#L622 

Added line #L622 was not covered by tests

Check warning on line 1026 in internal/graph/lookupresources3.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lookupresources3.go#L1026 

Added line #L1026 was not covered by tests

Check warning on line 551 in internal/graph/lookupsubjects.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lookupsubjects.go#L551 

Added line #L551 was not covered by tests

Check warning on line 172 in internal/graph/lr2streams.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/graph/lr2streams.go#L172 

Added line #L172 was not covered by tests

Check warning on line 176 in internal/namespace/canonicalization.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/namespace/canonicalization.go#L176 

Added line #L176 was not covered by tests

Check warning on line 264 in internal/namespace/canonicalization.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/namespace/canonicalization.go#L264 

Added line #L264 was not covered by tests

Check warning on line 21 in internal/namespace/caveats.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/namespace/caveats.go#L21 

Added line #L21 was not covered by tests

Check warning on line 23 in internal/namespace/caveats.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/namespace/caveats.go#L23 

Added line #L23 was not covered by tests

Check warning on line 53 in internal/namespace/caveats.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/namespace/caveats.go#L53 

Added line #L53 was not covered by tests

Check warning on line 176 in internal/relationships/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/relationships/errors.go#L176 

Added line #L176 was not covered by tests

Check warning on line 191 in internal/relationships/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/relationships/errors.go#L191 

Added line #L191 was not covered by tests

Check warning on line 164 in internal/services/shared/schema.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/services/shared/schema.go#L164 

Added line #L164 was not covered by tests

Check warning on line 57 in internal/services/v1/bulkcheck.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/services/v1/bulkcheck.go#L57 

Added line #L57 was not covered by tests

Check warning on line 200 in internal/services/v1/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/services/v1/errors.go#L200 

Added line #L200 was not covered by tests

Check warning on line 325 in internal/services/v1/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/services/v1/errors.go#L325 

Added line #L325 was not covered by tests
View more details on Codecov