Skip to content

feat: dispatch-parity quality improvements#34

Merged
jongio merged 3 commits intomainfrom
quality/dispatch-parity
Mar 16, 2026
Merged

feat: dispatch-parity quality improvements#34
jongio merged 3 commits intomainfrom
quality/dispatch-parity

Conversation

@jongio
Copy link
Copy Markdown
Owner

@jongio jongio commented Mar 15, 2026

Dispatch-Parity Quality Improvements

Brings azd-core CI/CD and code quality to parity with jongio/dispatch.

CI/CD & Security

  • SHA-pinned all GitHub Actions to immutable commit SHAs
  • Added CODEOWNERS with @jongio
  • Added CodeQL security scanning workflow
  • Added govulncheck vulnerability scanning workflow
  • Added Dependabot for Go modules and GitHub Actions
  • Added cosign code signing and SBOM generation to release workflow
  • Added concurrency control to CI workflows

Code Quality

  • Upgraded golangci-lint config to proper v2 format (30+ linters)
  • Added deadcode detection, gofumpt formatting, cross-GOOS linting to Magefile
  • Fixed 419 lint issues (doc comments, misspell, prealloc, errcheck, etc.)
  • Added README badges (CI, Release, Go Report Card, Go Reference)

Files Changed

24 files changed, 458 insertions(+), 138 deletions(-)

jongio and others added 2 commits March 15, 2026 06:44
Remove buzzword copy, obvious comments that restate adjacent code,
and generic filler text identified by anti-slop review.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove "Best-in-Class", "Leverage battle-tested", and obvious
restating comments in yamlutil and env packages.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jongio jongio force-pushed the quality/dispatch-parity branch 2 times, most recently from 3faeb76 to 1fb1559 Compare March 15, 2026 13:54
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 15, 2026

Codecov Report

❌ Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.04%. Comparing base (4e95aa0) to head (6c918f6).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
healthcheck/profiles.go 0.00% 0 Missing and 2 partials ⚠️
healthcheck/checker.go 0.00% 1 Missing ⚠️
httpclient/client.go 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   81.01%   81.04%   +0.02%     
==========================================
  Files          41       41              
  Lines        3803     3798       -5     
==========================================
- Hits         3081     3078       -3     
+ Misses        516      515       -1     
+ Partials      206      205       -1     
Flag Coverage Δ
unittests 81.04% <80.95%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jongio jongio force-pushed the quality/dispatch-parity branch from 1fb1559 to f8bd229 Compare March 15, 2026 20:16
- Pin all GitHub Actions to full commit SHAs
- Add CODEOWNERS file
- Add Dependabot for go modules and github-actions
- Add concurrency control to CI/PR workflows
- Add CodeQL security scanning workflow
- Add govulncheck vulnerability scanning workflow
- Standardize golangci-lint config with 30+ linters
- Add dispatch-level linters (errname, exhaustive, forcetypeassert, etc.)
- Add gofumpt strict formatting checks
- Add deadcode detection
- Add cosign code signing to release workflow
- Add SBOM generation (SPDX + CycloneDX) to release workflow
- Add comprehensive README badges (CI, CodeQL, Go Report Card, etc.)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jongio jongio force-pushed the quality/dispatch-parity branch from f8bd229 to 6c918f6 Compare March 15, 2026 20:23
@jongio jongio merged commit 0880335 into main Mar 16, 2026
6 checks passed
@jongio jongio deleted the quality/dispatch-parity branch March 16, 2026 16:10
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