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

ci(website): add change detection to skip unconditional hourly deploys #515

Merged
guettlibot merged 3 commits from issue-494-website-change-detection into main 2026-06-07 03:04:59 +00:00
Conversation 0 Commits 3 Files Changed 1
+471 -1152
guettlibot commented 2026-06-07 00:28:57 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Adds a check-changes job to website.yml that detects whether any website-relevant files changed since the last successful deploy, before running the expensive build.
  • On schedule (cron), queries the Forgejo API for the last successful website.yml run, compares head_sha to HEAD, then diffs for website/**, scripts/website-verify.sh, or .forgejo/workflows/website.yml paths.
  • On push or workflow_dispatch, always deploys (same behaviour as before).
  • The existing deploy job is unchanged in content; it now runs conditionally on needs.check-changes.outputs.has_changes == 'true'.

Modelled directly after the check-changes job already in deploy.yml.

Test plan

  • Verify that a cron run with no website changes shows has_changes=false and skips the deploy job
  • Verify that a cron run after a website commit shows has_changes=true and runs the deploy job
  • Verify that a manual workflow_dispatch always runs the deploy job
  • Verify that a push to website/** still runs the deploy job

Closes #494

🤖 Generated with Claude Code

## Summary - Adds a `check-changes` job to `website.yml` that detects whether any website-relevant files changed since the last successful deploy, before running the expensive build. - On `schedule` (cron), queries the Forgejo API for the last successful `website.yml` run, compares `head_sha` to `HEAD`, then diffs for `website/**`, `scripts/website-verify.sh`, or `.forgejo/workflows/website.yml` paths. - On `push` or `workflow_dispatch`, always deploys (same behaviour as before). - The existing `deploy` job is unchanged in content; it now runs conditionally on `needs.check-changes.outputs.has_changes == 'true'`. Modelled directly after the `check-changes` job already in `deploy.yml`. ## Test plan - [ ] Verify that a cron run with no website changes shows `has_changes=false` and skips the deploy job - [ ] Verify that a cron run after a website commit shows `has_changes=true` and runs the deploy job - [ ] Verify that a manual `workflow_dispatch` always runs the deploy job - [ ] Verify that a `push` to `website/**` still runs the deploy job Closes #494 🤖 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#515