Skip to content

ci: preserve conditional installer validation - #24

Merged
MrPointer merged 2 commits into
mainfrom
fix/pr-merge-stuck-without-ci
Aug 1, 2026
Merged

MrPointer merged 2 commits into
mainfrom
fix/pr-merge-stuck-without-ci

Conversation

@MrPointer

@MrPointer MrPointer commented Aug 1, 2026 •

Copy link
Copy Markdown
Owner

📝 Summary

Preserves installer-only build, test, and E2E validation while ensuring a required status check is reported for every pull request.
Adds lightweight path detection so unrelated pull requests are not blocked by an expected but never-created check.
Excludes work-specific Claude configuration from personal installs, preventing a missing personal.work_name from aborting chezmoi apply.

Key Decisions

Conditional installer validation

The change detector uses the existing installer and workflow path set. The costly validation chain runs only when that filter reports a matching change.

Required check reporting

Installer CI Status is created for every pull request, accepts intentionally skipped installer jobs, and fails when change detection or an applicable validation job fails.

Personal Claude configuration

The Claude shell wrapper and my-claude-code nono profile require work SSH key names, so personal installations ignore both files rather than rendering templates without personal.work_name.

🔄 Before & After

Area Before After
Unrelated pull requests Required CI check is expected but never reported Lightweight status gate reports success
Installer changes Full validation runs Full validation still runs
Personal installs Work-only Claude templates abort on missing work name Work-only Claude files are ignored
Pushes to main Path-filtered workflow Path-filtered workflow

Run a lightweight change detector and status gate for every pull request while preserving the installer-only build, test, and E2E scope.
@MrPointer
MrPointer enabled auto-merge (squash) August 1, 2026 17:45
Exclude Claude's work SSH-key wrapper and nono profile when the installer does not define personal.work_name.
@MrPointer
MrPointer merged commit 81edbf9 into main Aug 1, 2026
9 checks passed
@MrPointer
MrPointer deleted the fix/pr-merge-stuck-without-ci branch September 6, 2026 15:39
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