Skip to content

ci: modernize Go tooling and workflows - #12

Merged
madflojo merged 1 commit into
americanexpress:mainfrom
madflojo:feature/modernize-tooling-ci
Jun 18, 2026
Merged

ci: modernize Go tooling and workflows#12
madflojo merged 1 commit into
americanexpress:mainfrom
madflojo:feature/modernize-tooling-ci

Conversation

@madflojo

Copy link
Copy Markdown
Member

Summary

Modernizes the project tooling and CI while keeping the existing Coveralls reporting. This also adds local plan ignores so planning files stay out of future diffs. Cleaner workshop, fewer loose nails.

Changes

  • Add .plans/ to .gitignore for local planning notes.
  • Update go.mod to Go 1.24.
  • Expand Makefile targets for build, tests, lint, coverage, format, clean, benchmarks, and race benchmarks.
  • Refresh test and lint workflows with current pinned actions and go-version-file.
  • Keep Coveralls for coverage publishing.
  • Add a benchmark workflow using benchmark-action/github-action-benchmark for non-race benchmarks.
  • Run race-enabled benchmarks as part of the standard tests workflow.

Validation

  • make build
  • make tests
  • make lint
  • make coverage
  • make benchmarks
  • make benchmarks-race

Risks

  • CI now requires Go 1.24.
  • Benchmark tracking depends on the benchmark action cache path being available across workflow runs.

Refresh local targets and CI actions, keep Coveralls, add local plan ignore, and separate benchmark runs. Sharper tools, fewer splinters.
@madflojo
madflojo force-pushed the feature/modernize-tooling-ci branch from 2b63edd to 918e932 Compare June 18, 2026 13:57
@madflojo
madflojo merged commit bfdb62b into americanexpress:main Jun 18, 2026
4 checks passed
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