Skip to content

fix: align toolchain versions and remove disabled/duplicate CI files#32

Merged
Patel230 merged 3 commits into
mainfrom
fix/production-readiness-audit
Jun 9, 2026
Merged

fix: align toolchain versions and remove disabled/duplicate CI files#32
Patel230 merged 3 commits into
mainfrom
fix/production-readiness-audit

Conversation

@Patel230

@Patel230 Patel230 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Production-readiness fixes for the trace repo: align toolchain versions and remove disabled/duplicate CI files.

Changes

  • mise.toml: bumped go 1.26.1 -> 1.26.4 to match go.mod/CI.
  • CI: aligned golangci-lint version between ci.yml and mise.toml; pinned gofumpt v0.10.0 (was @latest).
  • Removed two permanently-disabled workflow stubs:
    • license-check.yml -> referenced a non-existent shared workflow.
    • lint.yml -> redundant with ci.yml.
  • Removed duplicate .github/CODEOWNERS (kept the richer root CODEOWNERS).

Verification

  • go build / go vet / go test pass.
  • gofumpt clean.
  • golangci-lint reports 0 issues.

Patel230 added 3 commits June 9, 2026 11:56
- mise.toml: bumped go 1.26.1 -> 1.26.4 to match go.mod/CI.
- CI: aligned golangci-lint version between ci.yml and mise.toml; pinned gofumpt v0.10.0 (was @latest).
- Removed two permanently-disabled workflow stubs (license-check.yml -> non-existent shared workflow; lint.yml -> redundant with ci.yml).
- Removed duplicate .github/CODEOWNERS (kept the richer root CODEOWNERS).

Verified: go build/vet/test pass; gofumpt clean; golangci-lint 0 issues.
The .github/CODEOWNERS (`* @GrayCodeAI/cli-maintainers`) took GitHub
precedence over the richer root CODEOWNERS, shadowing the per-directory
ownership (security-team for /redact/, devops-team for CI/build, etc.).
Removing it restores the intended ownership policy.
@Patel230 Patel230 merged commit 9b64e0b into main Jun 9, 2026
14 checks passed
@Patel230 Patel230 deleted the fix/production-readiness-audit branch June 9, 2026 11:11
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