Skip to content

Apply lint fixes for oss-foundations#10796

Open
stephanos wants to merge 12 commits into
mainfrom
stephanos/split-oss-foundations
Open

Apply lint fixes for oss-foundations#10796
stephanos wants to merge 12 commits into
mainfrom
stephanos/split-oss-foundations

Conversation

@stephanos

@stephanos stephanos commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

What changed?

  • Applied testifylint, staticcheck, and gofix auto-fixes.
  • Exact commands that were run:
.bin/golangci-lint-v2.9.0 run --allow-parallel-runners --concurrency 4 --fix --enable-only testifylint --build-tags disable_grpc_modules,test_dep --timeout 20m --config=.github/.golangci.yml
.bin/golangci-lint-v2.9.0 run --allow-parallel-runners --concurrency 4 --fix --enable-only staticcheck --build-tags disable_grpc_modules,test_dep --timeout 20m --config=.github/.golangci.yml
make fmt-gofix
make goimports
make fmt
git diff --check
  • No manual or AI changes were made; except where commented on.
  • Some fixes caused lint errors; those were reverted again.
  • Changes were all reviewed by me.

@stephanos stephanos changed the title Extract OSS Foundations lint auto-fixes Apply lint fixes for oss-foundations Jun 21, 2026
@stephanos stephanos marked this pull request as ready for review June 21, 2026 18:40
@stephanos stephanos requested a review from a team as a code owner June 21, 2026 18:40
@stephanos stephanos marked this pull request as draft June 21, 2026 19:34
}
}

func timePtr(t time.Time) *time.Time {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Manual change; inlined this

@stephanos stephanos marked this pull request as ready for review June 21, 2026 22:08
@stephanos stephanos requested review from a team as code owners June 21, 2026 22:08
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