Skip to content

Lint Roller 13: integration-tests - #23505

Open
kalverra wants to merge 4 commits into
DX-5066/12-deployment-part2-devenvfrom
DX-5066/13-integration-tests
Open

Lint Roller 13: integration-tests#23505
kalverra wants to merge 4 commits into
DX-5066/12-deployment-part2-devenvfrom
DX-5066/13-integration-tests

Conversation

@kalverra

Copy link
Copy Markdown
Collaborator

Issues Fixed

Linter Issues Fixed
testifylint 94
revive 41
prealloc 21
gocritic 10
staticcheck 4
Total 170

Packages Covered

  • integration-tests/smoke/ccip
  • integration-tests/...
  • integration-tests/load

Stack created with GitHub Stacks CLIGive Feedback 💬

@github-actions

github-actions Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

CORA - Pending Reviewers

Codeowners Entry Overall Num Files Owners
* 1 @smartcontractkit/foundations, @smartcontractkit/core
/integration-tests/ 11 @smartcontractkit/devex-tooling, @smartcontractkit/core
/integration-tests/**/*ccip* 23 @smartcontractkit/ccip-offchain, @smartcontractkit/core, @smartcontractkit/ccip

Legend: ✅ Approved | ❌ Changes Requested | 💬 Commented | 🚫 Dismissed | ⏳ Pending | ❓ Unknown

For more details, see the full review summary.

@kalverra
kalverra force-pushed the DX-5066/13-integration-tests branch from db7d340 to 7c676d6 Compare August 24, 2026 21:12
@kalverra
kalverra requested review from a team as code owners August 25, 2026 17:45
@kalverra
kalverra force-pushed the DX-5066/13-integration-tests branch from 7c676d6 to 5f20e04 Compare August 25, 2026 17:45
@kalverra
kalverra force-pushed the DX-5066/13-integration-tests branch from 5f20e04 to 80fba6e Compare August 25, 2026 17:53
kalverra and others added 4 commits August 26, 2026 15:34
Fix errcheck, staticcheck (SA1019 deprecated ccipocr3/bind.Object),
testifylint, prealloc, goimports, and ineffassign violations in
integration-tests/smoke/ccip.
CreateJobDistributor call used the stale Uri field name; the struct
field is URI, breaking typecheck for the core/scripts module.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Fix gocritic, perfsprint, prealloc, revive, staticcheck, and unconvert
violations in integration-tests/... (actions, docker/test_env,
testconfig, testsetups/ccip, types, utils).
Fix testifylint (go-require), unused, gocritic, prealloc, and
staticcheck (SA1019 deprecated ccipocr3/AddressBook) violations in
integration-tests/load/ccip.
@kalverra
kalverra force-pushed the DX-5066/13-integration-tests branch from 80fba6e to 92c83b4 Compare August 26, 2026 19:36
@cl-sonarqube-production

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.9% Duplication on New Code

See analysis details on SonarQube

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