Skip to content

CHORE: Split CI into separate test and rubocop jobs - #2

Merged
hahuang65 merged 1 commit into
mainfrom
hh/ci
May 29, 2026
Merged

CHORE: Split CI into separate test and rubocop jobs#2
hahuang65 merged 1 commit into
mainfrom
hh/ci

Conversation

@hahuang65

Copy link
Copy Markdown
Member

What: Replaces the single build job that ran bundle exec rake
with two parallel jobs — test (runs bundle exec rake test) and
rubocop (runs bundle exec rake rubocop). Workflow renamed
Ruby -> CI to reflect the broader scope.

Why: Separate jobs give clearer signal in PR check lists
(test/rubocop failures are distinct red Xs instead of one combined
status), and they execute in parallel — faster feedback per PR.

Also adds:

  • concurrency block keyed on workflow + ref, with
    cancel-in-progress so a force-push to a PR cancels the previous
    in-flight run instead of doubling CI usage.
  • fail-fast: false on the test matrix so a single Ruby version
    failure doesn't cancel siblings (useful once the matrix grows
    past the current single 4.0.5 entry).

Same Ruby version (4.0.5), same actions/checkout@v6 and
ruby/setup-ruby@v1 with bundler-cache enabled.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

What: Replaces the single `build` job that ran `bundle exec rake`
with two parallel jobs — `test` (runs `bundle exec rake test`) and
`rubocop` (runs `bundle exec rake rubocop`). Workflow renamed
Ruby -> CI to reflect the broader scope.

Why: Separate jobs give clearer signal in PR check lists
(test/rubocop failures are distinct red Xs instead of one combined
status), and they execute in parallel — faster feedback per PR.

Also adds:
- concurrency block keyed on workflow + ref, with
  cancel-in-progress so a force-push to a PR cancels the previous
  in-flight run instead of doubling CI usage.
- fail-fast: false on the test matrix so a single Ruby version
  failure doesn't cancel siblings (useful once the matrix grows
  past the current single 4.0.5 entry).

Same Ruby version (4.0.5), same actions/checkout@v6 and
ruby/setup-ruby@v1 with bundler-cache enabled.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@hahuang65
hahuang65 merged commit 2cb131f into main May 29, 2026
2 checks passed
@hahuang65
hahuang65 deleted the hh/ci branch May 29, 2026 06:07
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