Skip to content

Skip expensive test setup for documentation-only changes - #187

Closed
dblock wants to merge 1 commit into
slack-ruby:masterfrom
dblock:ci/docs-only-tests
Closed

dblock wants to merge 1 commit into
slack-ruby:masterfrom
dblock:ci/docs-only-tests

Conversation

@dblock

@dblock dblock commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

Closes #166. Classify changed files in a shared reusable workflow. For Markdown, license, and documentation-image-only changes, preserve every existing matrix check name but skip checkout, database containers, Ruby/dependency setup, Firefox, and tests in those jobs. Empty service images disable database startup. Danger and RuboCop stay unchanged; all 22 database compatibility combinations and their names stay intact.

Use merge-base comparisons for pull requests (including forks), before/after comparisons for pushes, and NUL-delimited filenames with rename detection disabled so deleted or renamed code cannot look docs-only. New branches and unavailable old push revisions explicitly run the full matrix. Invalid PR comparisons fail visibly rather than silently skipping tests. No path-level workflow filtering and no additional permissions or external filtering action.

Validation

actionlint passes for all three workflows; focused RuboCop passes. Eight classifier specs exercise real temporary Git repositories for docs-only and mixed changes, code-to-doc renames, PR base divergence, history-rewriting pushes, new branches, missing old revisions, and invalid comparisons. Service suppression and check publication still need the hosted workflow run.

Authored by Copilot (AI), on behalf of @dblock.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the ci/docs-only-tests branch from 59f0977 to fd3553b Compare October 25, 2026 20:06
@dblock dblock closed this Oct 25, 2026
@github-actions

github-actions Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

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.

Prevent test workflows from running when updating docs only

1 participant