Skip to content

ci: enable homeboy-action test-scope changed for PRs#613

Merged
chubes4 merged 7 commits intomainfrom
chore/pr-test-scope-wiring
Mar 5, 2026
Merged

ci: enable homeboy-action test-scope changed for PRs#613
chubes4 merged 7 commits intomainfrom
chore/pr-test-scope-wiring

Conversation

@chubes4
Copy link
Member

@chubes4 chubes4 commented Mar 4, 2026

Depends on:

Summary

  • set test-scope: 'changed' in data-machine Homeboy PR workflow
  • keeps PR checks aligned with policy: changed-scope lint/test/audit

Note

This wiring expects Homeboy test command support for --changed-since (tracked in #447). If that support is absent in the installed Homeboy release, this PR will fail and should remain blocked until #447 lands.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

Homeboy Results — data-machine

Tooling versions

  • Homeboy CLI: homeboy 0.56.0
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: unknown
  • Action: Extra-Chill/homeboy-action@v1

ℹ️ PR test scope resolved to full for compatibility with installed Homeboy CLI

lint (changed files only)

  • PHPStan: PHPSTAN SUMMARY: 133 errors at level 5

Homeboy Action v1 — homeboy 0.56.0

@chubes4
Copy link
Member Author

chubes4 commented Mar 5, 2026

Updated this PR based on current capability reality:

What I changed

  • Added push trigger on main so update/version-bump runs happen automatically
  • Enabled auto-issue: true for non-PR failures on main
  • Set test-scope: full for now

Why test-scope changed is not enabled yet

homeboy-action wires changed-scope by passing --changed-since to homeboy test, but current homeboy test CLI does not support that flag yet (reproduced: Unknown option --changed-since).

Opened tracking issue:

So this PR now keeps CI green/reliable while still delivering the main-branch full-suite + auto-issue behavior you requested.

@chubes4
Copy link
Member Author

chubes4 commented Mar 5, 2026

Updated this PR again to mirror Homeboy repo structure:\n\n- split CI into separate jobs:\n - (lint,test)\n - (audit only)\n- both jobs run on PR and push-to-main\n- now only on non-PR runs ()\n\nThis keeps PR signal clearer and preserves main-branch issue automation for update/version-bump runs.

@chubes4
Copy link
Member Author

chubes4 commented Mar 5, 2026

Correction (previous comment had shell interpolation artifacts).

Updated this PR again to mirror Homeboy repo structure:

  • split CI into separate jobs:
    • build (lint,test)
    • audit (audit only)
  • both jobs run on PR and push-to-main
  • auto-issue now only on non-PR runs (github.event_name != pull_request)

This keeps PR signal clearer and preserves main-branch issue automation for update/version-bump runs.

chubes4 added 3 commits March 5, 2026 14:35
Set homeboy-action test-scope to changed so PR checks run lint/test/audit on changed impact scope while release workflows remain full-suite.
@chubes4 chubes4 force-pushed the chore/pr-test-scope-wiring branch from 82e8638 to a570f21 Compare March 5, 2026 14:36
@chubes4 chubes4 merged commit a246a86 into main Mar 5, 2026
2 checks passed
@chubes4 chubes4 deleted the chore/pr-test-scope-wiring branch March 5, 2026 14:54
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