Skip to content

chore: lint fixes part 3#255

Merged
pawbana merged 1 commit intomainfrom
pb/coder-lint-fix3
Apr 13, 2026
Merged

chore: lint fixes part 3#255
pawbana merged 1 commit intomainfrom
pb/coder-lint-fix3

Conversation

@pawbana
Copy link
Copy Markdown
Contributor

@pawbana pawbana commented Apr 10, 2026 •
edited
Loading

  • receiver-naming (7): standardized receiver names to be consistent within each type (i for interception types)
  • unused-receiver (34): omitted unused receiver names on methods that don't reference the receiver (e.g. Streaming(), Type(), AuthHeader(), encodeForStream(), etc.)
  • unused-parameter (11): renamed unused ctx to _ in mock recorders, MCP proxy shutdown, and async recorder; renamed unused w to _ in CreateInterceptor; renamed unused ctx to _ in mock MCP tool callback
  • unused field (1): removed unused closeOnce sync.Once field from eventstream

@pawbana pawbana mentioned this pull request Apr 10, 2026
Merged
@pawbana Graphite App
Copy link
Copy Markdown
Contributor Author

pawbana commented Apr 10, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Apr 10, 2026
Merged
Merged
@pawbana pawbana force-pushed the pb/coder-lint-fix3 branch from 9f6f8fe to 484f365 Compare April 10, 2026 12:30
@pawbana pawbana force-pushed the pb/coder-lint-fix2 branch from 6c82ff7 to 3f49be8 Compare April 10, 2026 12:30
@pawbana pawbana force-pushed the pb/coder-lint-fix3 branch from 484f365 to ead0336 Compare April 10, 2026 12:30
This was referenced Apr 10, 2026
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@pawbana pawbana marked this pull request as ready for review April 10, 2026 22:12
@pawbana pawbana mentioned this pull request Apr 13, 2026
Merged
@pawbana pawbana force-pushed the pb/coder-lint-fix3 branch from ead0336 to e63a56f Compare April 13, 2026 11:30
@pawbana pawbana force-pushed the pb/coder-lint-fix2 branch from 3f49be8 to 5c0d4f3 Compare April 13, 2026 11:30
@pawbana Graphite App
Copy link
Copy Markdown
Contributor Author

pawbana commented Apr 13, 2026 •
edited
Loading

Merge activity

  • Apr 13, 3:10 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 13, 3:13 PM UTC: Graphite couldn't merge this pull request because a downstack PR chore: lint fixes part 1 (automatic fixes) #253 failed to merge.
  • Apr 13, 3:24 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 13, 3:27 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 13, 3:29 PM UTC: @pawbana merged this pull request with Graphite.

@pawbana pawbana force-pushed the pb/coder-lint-fix2 branch from 5c0d4f3 to 80c1328 Compare April 13, 2026 15:22
@pawbana pawbana force-pushed the pb/coder-lint-fix3 branch from e63a56f to 872a966 Compare April 13, 2026 15:22
@pawbana pawbana changed the base branch from pb/coder-lint-fix2 to graphite-base/255 April 13, 2026 15:25
@pawbana pawbana changed the base branch from graphite-base/255 to main April 13, 2026 15:26
@pawbana pawbana force-pushed the pb/coder-lint-fix3 branch from 872a966 to 798a176 Compare April 13, 2026 15:27
@pawbana pawbana merged commit e892e92 into main Apr 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dannykopping dannykopping dannykopping approved these changes

@ssncferreira ssncferreira Awaiting requested review from ssncferreira

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pawbana @dannykopping