Skip to content

ci: workflow and pipeline improvements#247

Merged
jongio merged 1 commit into
mainfrom
patrol/ci-workflow-fixes
May 18, 2026
Merged

ci: workflow and pipeline improvements#247
jongio merged 1 commit into
mainfrom
patrol/ci-workflow-fixes

Conversation

@jongio
Copy link
Copy Markdown
Owner

@jongio jongio commented May 18, 2026

This PR addresses 7 CI/workflow issues:

  • Add push trigger for main branch - CI now runs on pushes to main, not just PRs
  • Remove duplicate setup/build in release test job - Eliminated redundant Go/Node/pnpm setup and dashboard rebuild after tests
  • Narrow linter exclusions - Removed overly broad exclusions (errcheck, gosec, staticcheck, etc.) from test files, keeping only truly irrelevant ones (dupl, goconst, unparam, prealloc, noctx, exhaustive)
  • Enable race detector on macOS - Removed the macOS conditional skip; race detection now runs on all platforms with CGO_ENABLED=1
  • Pin tool versions - golangci-lint v2.1.6, staticcheck v0.6.1, gosec v2.22.4, govulncheck v1.1.4, mage v1.15.0
  • Add npm ecosystem to Dependabot - Added entries for cli/dashboard and web directories
  • Consolidate dashboard builds - Dashboard is now built once in a dedicated job and shared via artifacts to test/lint/build/integration jobs

Fixes #225
Fixes #224
Fixes #221
Fixes #220
Fixes #218
Fixes #217
Fixes #209

- Add push trigger for main branch to CI workflow (#225)
- Remove duplicate setup and dashboard build steps in release test job (#224)
- Narrow linter exclusions to only skip truly irrelevant checks in test files (#221)
- Enable race detector on macOS CI runs with CGO_ENABLED=1 (#220)
- Pin tool versions (golangci-lint v2.1.6, gosec v2.22.4, etc.) (#218)
- Add npm ecosystem entries for cli/dashboard and web to dependabot (#217)
- Consolidate dashboard builds into single job with artifact sharing (#209)
@jongio jongio merged commit 973a21d into main May 18, 2026
9 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment