Skip to content

ci(lint): refactor test assertions and linter configuration#4436

Merged
appleboy merged 1 commit intomasterfrom
lint2
Nov 15, 2025
Merged

ci(lint): refactor test assertions and linter configuration#4436
appleboy merged 1 commit intomasterfrom
lint2

Conversation

@appleboy
Copy link
Copy Markdown
Member

  • Update golangci-lint GitHub Action version from v2.1.6 to v2.6
  • Remove the gci formatter and exclusions for third_party, builtin, and examples from the linter config
  • Fix argument order for assert.EqualValues and assert.Exactly in context tests for clarity
  • Refactor integration tests to build response strings using strings.Builder instead of direct concatenation for improved performance and readability

- Update golangci-lint GitHub Action version from v2.1.6 to v2.6
- Remove the gci formatter and exclusions for third_party, builtin, and examples from the linter config
- Fix argument order for assert.EqualValues and assert.Exactly in context tests for clarity
- Refactor integration tests to build response strings using strings.Builder instead of direct concatenation for improved performance and readability

Signed-off-by: appleboy <appleboy.tw@gmail.com>
@appleboy appleboy added this to the v1.12 milestone Nov 15, 2025
@appleboy appleboy merged commit fb27ef2 into master Nov 15, 2025
25 checks passed
@appleboy appleboy deleted the lint2 branch November 15, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant