Skip to content

Add optional project_stats input to rails-ci workflow - #27

Open
wesrich wants to merge 4 commits into
mainfrom
claude/project-stats-ci-integration-234e1e
Open

Add optional project_stats input to rails-ci workflow#27
wesrich wants to merge 4 commits into
mainfrom
claude/project-stats-ci-integration-234e1e

Conversation

@wesrich

@wesrich wesrich commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds an optional project_stats boolean input to the rails-ci.yml reusable workflow (default false)
  • When enabled, runs the project-stats action as a background: true step right after gems are installed, passing install-ruby: false so it reuses the workflow's existing Ruby setup instead of installing it a second time
  • Nothing waits on the step, so enabling it adds no wall-clock time to CI; results land in the job summary whenever the background step finishes

Test plan

  • Set project_stats: true in a caller repo's workflow and confirm the Project Stats table shows up in the job summary without lengthening the CI job

@wesrich wesrich self-assigned this Aug 11, 2026
Runs the project-stats action in the background right after gems are
installed, reusing the workflow's existing Ruby setup instead of
installing it a second time, so nothing waits on it and stats land in
the job summary for free.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@wesrich
wesrich force-pushed the claude/project-stats-ci-integration-234e1e branch from 54edbee to 53b7d97 Compare August 11, 2026 14:55
wesrich and others added 3 commits August 11, 2026 13:28
RoleModel/actions/project-stats was added in v3.1.0, so it doesn't
exist at v1 — pin to v3 instead, matching the rest of the workflow.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Gives project_stats its own entry in the PR checks list, matching
Test Results/brakeman, instead of being buried inside the ci job's
step log. The stats table is still captured early as a background
step (right after gems install) for zero added wall-clock time; only
the lightweight Check Run publish is deferred to the end, grouped
with the other end-of-run reporting steps.

project-stats/action.yml keeps writing to $GITHUB_STEP_SUMMARY too,
so its existing standalone usage (its own job, own summary) is
unaffected — it now also exposes the table as a `table` output for
callers that want to publish it elsewhere.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Self-reference was still pinned to the released v3 tag, which doesn't
have the new `table` output yet. Will revert to @V3 before merge.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant