ci: clean up workflow #3

Merged
mvhutz merged 10 commits from cli/clean-up into main 2026-03-18 00:47:02 +00:00
Owner

Description

The current CI workflow was auto-generated by Claude, and is really not intuitive. Fixing it up to make it work better.

  • Add names for jobs in workflow.
  • Rename workflow to ci.yml.
  • Added go mod tidy as a tasks in the lint job.

Decisions

  • Chose not to use emojis for workflow or job names. While they look nice, they are unprofessional, and harder to grep.
## Description The current CI workflow was auto-generated by Claude, and is really not intuitive. Fixing it up to make it work better. - Add names for jobs in workflow. - Rename workflow to `ci.yml`. - Added `go mod tidy` as a tasks in the `lint` job. ### Decisions - Chose not to use emojis for workflow or job names. While they look nice, they are unprofessional, and harder to `grep`.
mvhutz added 2 commits 2026-03-17 23:53:35 +00:00
The `lint.yml` workflow contains more than just linting jobs. Renaming it to `ci.yml`, which is more general, and common practice.
feat: only one CI job at a time per PR
All checks were successful
CI / lint (pull_request) Successful in 52s
CI / unit-test (pull_request) Successful in 26s
CI / fuzz-test (pull_request) Successful in 1m3s
CI / mutation-test (pull_request) Successful in 43s
9726670dea
This prevents the Gitea runner from getting clogged up with stale CI checks.
mvhutz added 1 commit 2026-03-17 23:54:21 +00:00
test: does the previous CI check get canceled?
All checks were successful
CI TEST / lint (pull_request) Successful in 51s
CI TEST / unit-test (pull_request) Successful in 26s
CI TEST / fuzz-test (pull_request) Successful in 1m3s
CI TEST / mutation-test (pull_request) Successful in 2m37s
a9b157d4b2
mvhutz added 1 commit 2026-03-18 00:03:47 +00:00
test: try again, doubled the job capacity
All checks were successful
CI TEST2 / unit-test (pull_request) Successful in 55s
CI TEST2 / lint (pull_request) Successful in 1m43s
CI TEST2 / fuzz-test (pull_request) Successful in 1m12s
CI TEST2 / mutation-test (pull_request) Successful in 1m1s
f31100b23a
mvhutz added 1 commit 2026-03-18 00:04:16 +00:00
test: will this cancel?
Some checks failed
CI TEST3 / lint (pull_request) Successful in 1m9s
CI TEST3 / unit-test (pull_request) Successful in 36s
CI TEST3 / mutation-test (pull_request) Successful in 1m4s
CI TEST3 / fuzz-test (pull_request) Failing after 55s
7148704dd3
mvhutz added 1 commit 2026-03-18 00:10:58 +00:00
revert: remove concurrency from gha
All checks were successful
CI TEST3 / unit-test (pull_request) Successful in 37s
CI TEST3 / lint (pull_request) Successful in 1m2s
CI TEST3 / mutation-test (pull_request) Successful in 1m12s
CI TEST3 / fuzz-test (pull_request) Successful in 1m16s
2faa8feedc
The `cancel-in-progress` option for concurrency is not really supported by Gitea, so the whole point of the block no longer makes any sense.
mvhutz changed title from ci: clean up workflow to wip ci: clean up workflow 2026-03-18 00:16:38 +00:00
mvhutz changed title from wip ci: clean up workflow to WIP: wip ci: clean up workflow 2026-03-18 00:16:49 +00:00
mvhutz changed title from WIP: wip ci: clean up workflow to WIP: ci: clean up workflow 2026-03-18 00:16:56 +00:00
mvhutz added 1 commit 2026-03-18 00:17:52 +00:00
revert: keep workflow title as "CI"
All checks were successful
CI / unit-test (pull_request) Successful in 37s
CI / lint (pull_request) Successful in 1m2s
CI / fuzz-test (pull_request) Successful in 1m15s
CI / mutation-test (pull_request) Successful in 1m10s
6a9505316a
mvhutz added 1 commit 2026-03-18 00:27:27 +00:00
style: add human-readable job names
All checks were successful
CI / Unit Tests (pull_request) Successful in 34s
CI / Golang Lint (pull_request) Successful in 1m0s
CI / Fuzz Tests (pull_request) Successful in 1m15s
CI / Mutation Tests (pull_request) Successful in 1m11s
334e594834
mvhutz added 1 commit 2026-03-18 00:36:29 +00:00
feat: add go mod tidy check job
Some checks failed
CI / Golang Lint (pull_request) Failing after 8s
CI / Unit Tests (pull_request) Successful in 30s
CI / Fuzz Tests (pull_request) Successful in 1m22s
CI / Mutation Tests (pull_request) Successful in 2m25s
b23f7226e3
mvhutz added 1 commit 2026-03-18 00:38:54 +00:00
fix: tidy up go.sum
All checks were successful
CI / Golang Lint (pull_request) Successful in 57s
CI / Unit Tests (pull_request) Successful in 36s
CI / Fuzz Tests (pull_request) Successful in 1m12s
CI / Mutation Tests (pull_request) Successful in 1m59s
65de0bd6af
The `go mod tidy` job caught something!
mvhutz changed title from WIP: ci: clean up workflow to ci: clean up workflow 2026-03-18 00:40:38 +00:00
mvhutz merged commit 0017a33f12 into main 2026-03-18 00:47:02 +00:00
mvhutz deleted branch cli/clean-up 2026-03-18 00:47:02 +00:00
mvhutz referenced this issue from a commit 2026-03-18 00:47:04 +00:00
Sign in to join this conversation.