Skip to content

feat: add GitHub Actions CI and perf:ignore comment support#1

Merged
cschuman merged 1 commit into
mainfrom
feat/session-20260105-211438
Jan 6, 2026
Merged

feat: add GitHub Actions CI and perf:ignore comment support#1
cschuman merged 1 commit into
mainfrom
feat/session-20260105-211438

Conversation

@cschuman

@cschuman cschuman commented Jan 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add comprehensive GitHub Actions CI workflow
  • Add // perf:ignore comment support for suppressing false positives

CI/CD

  • Multi-version Go testing (1.21, 1.22, 1.23)
  • golangci-lint integration
  • Self-audit (dogfooding goperf on itself)
  • Cross-platform binary builds (Linux, macOS, Windows)

Ignore Comments

// perf:ignore           - Ignore next line
// perf:ignore sql-in-loop - Ignore specific rule
// perf:ignore-start     - Start ignore block
// perf:ignore-end       - End ignore block

Test Plan

  • All 6 ignore tests pass
  • Build succeeds
  • Self-audit passes

CI/CD:
- Add comprehensive GitHub Actions workflow with:
  - Multi-version Go testing (1.21, 1.22, 1.23)
  - golangci-lint integration
  - Self-audit (dogfooding goperf on itself)
  - Cross-platform binary builds

Ignore comments:
- Add // perf:ignore for line-level suppression
- Add // perf:ignore rule-name for rule-specific suppression
- Add // perf:ignore-start / // perf:ignore-end for block suppression
- Full test coverage for ignore parsing
@cschuman cschuman merged commit c6cbab7 into main Jan 6, 2026
4 of 5 checks passed
@cschuman cschuman deleted the feat/session-20260105-211438 branch January 6, 2026 02:16
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