Skip to content

Lint Roller 12: deployment (keystone), devenv - #23504

Open
kalverra wants to merge 3 commits into
DX-5066/11-deployment-part1from
DX-5066/12-deployment-part2-devenv
Open

Lint Roller 12: deployment (keystone), devenv#23504
kalverra wants to merge 3 commits into
DX-5066/11-deployment-part1from
DX-5066/12-deployment-part2-devenv

Conversation

@kalverra

Copy link
Copy Markdown
Collaborator

Issues Fixed

Linter Issues Fixed
testifylint 88
revive 34
prealloc 20
gocritic 11
staticcheck 4
Total 157

Packages Covered

  • deployment/keystone/changeset
  • deployment/...
  • devenv

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
/devenv/ 7 @smartcontractkit/devex-tooling
/devenv/products/ocr2/ 1 @smartcontractkit/devex-tooling
/deployment/ 🚫 19 @smartcontractkit/ccip-tooling, @smartcontractkit/ccip-offchain, @smartcontractkit/keystone, @smartcontractkit/operations-platform, @smartcontractkit/core
/deployment/ccip/ 🚫 3 @smartcontractkit/ccip-tooling, @smartcontractkit/ccip-offchain, @smartcontractkit/operations-platform, @smartcontractkit/core
/deployment/common/ 🚫 4 @smartcontractkit/operations-platform
/deployment/cre/ 🚫 11 @smartcontractkit/keystone, @smartcontractkit/operations-platform
/deployment/data-feeds/ 🚫 5 @smartcontractkit/data-feeds-engineers, @smartcontractkit/operations-platform, @smartcontractkit/core
/deployment/keystone/ 🚫 32 @smartcontractkit/keystone, @smartcontractkit/operations-platform, @smartcontractkit/core

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

For more details, see the full review summary.

@github-actions

github-actions Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@trunk-io

trunk-io Bot commented Aug 23, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
TestConnectionManager_ReadyForTrafficRequiresEveryShard The test failed because a node address was unexpected, indicating a mismatch or misconfiguration in the network setup. Logs ↗︎

View Full Report ↗︎Docs

@kalverra
kalverra force-pushed the DX-5066/12-deployment-part2-devenv branch from c86a6fe to a63e8b7 Compare August 24, 2026 18:24
@kalverra
kalverra force-pushed the DX-5066/12-deployment-part2-devenv branch from a63e8b7 to 0dc0eef Compare August 24, 2026 21:07
@kalverra
kalverra force-pushed the DX-5066/12-deployment-part2-devenv branch from 0dc0eef to 371e278 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/12-deployment-part2-devenv branch from 371e278 to 91fbef5 Compare August 25, 2026 17:45
@kalverra
kalverra force-pushed the DX-5066/12-deployment-part2-devenv branch from 91fbef5 to 5404841 Compare August 25, 2026 17:53
@cl-sonarqube-production

Copy link
Copy Markdown

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.

2 participants