guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

ci: print explicit reason when deploy jobs are skipped #357

Merged
guettlibot merged 1 commits from refs/pull/357/head into main 2026-06-03 11:27:30 +00:00
Conversation 0 Commits 1 Files Changed 1
+1588 -9579
guettlibot commented 2026-06-03 11:16:36 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • The `Detect Changed Files` step in `deploy.yml` previously set `android=false` / `linux=false` silently, leaving downstream jobs showing only "skipped" in CI with no visible cause
  • Now each decision emits a clear one-liner in the step log:
    • `Android deploy: SKIPPED (no android-relevant files changed)`
    • `Android deploy: TRIGGERED (android-relevant files changed)`
    • `Linux deploy: SKIPPED (no linux-relevant files changed)`
    • or `HEAD already successfully deployed — skipping all deploy jobs`
  • The skip reason is visible in the `check-changes` job output, which is the job that makes the decision

Closes #353

Test plan

  • Trigger the deploy workflow on a commit that only touches CI/docs files — `check-changes` step log should show "Android deploy: SKIPPED (no android-relevant files changed)"
  • Trigger the deploy workflow on a commit touching `lib/` — log should show "Android deploy: TRIGGERED"
  • Trigger a second run on the same commit — log should show "already successfully deployed — skipping all deploy jobs"

🤖 Generated with Claude Code

## Summary - The \`Detect Changed Files\` step in \`deploy.yml\` previously set \`android=false\` / \`linux=false\` silently, leaving downstream jobs showing only "skipped" in CI with no visible cause - Now each decision emits a clear one-liner in the step log: - \`Android deploy: SKIPPED (no android-relevant files changed)\` - \`Android deploy: TRIGGERED (android-relevant files changed)\` - \`Linux deploy: SKIPPED (no linux-relevant files changed)\` - or \`HEAD <sha> already successfully deployed — skipping all deploy jobs\` - The skip reason is visible in the \`check-changes\` job output, which is the job that makes the decision Closes #353 ## Test plan - [ ] Trigger the deploy workflow on a commit that only touches CI/docs files — \`check-changes\` step log should show "Android deploy: SKIPPED (no android-relevant files changed)" - [ ] Trigger the deploy workflow on a commit touching \`lib/\` — log should show "Android deploy: TRIGGERED" - [ ] Trigger a second run on the same commit — log should show "already successfully deployed — skipping all deploy jobs" 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#357