Skip to content

GH#139: ci: skip locale planning when scheduled docs are unchanged#140

Merged
superdav42 merged 1 commit into
mainfrom
feature/auto-20260617-234516-gh139
Jun 18, 2026
Merged

GH#139: ci: skip locale planning when scheduled docs are unchanged#140
superdav42 merged 1 commit into
mainfrom
feature/auto-20260617-234516-gh139

Conversation

@superdav42

@superdav42 superdav42 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Added the same scheduled-run skip guard to the plan-locale-chunks workflow job that prepare-docs-source and build-locale-chunk already use, so unchanged scheduled runs do not spend CI minutes planning locale chunks.

Files Changed

.github/workflows/deploy-docs.yml

Runtime Testing

  • Risk level: Low (agent prompts / infrastructure scripts)
  • Verification: actionlint unavailable; verified the expected plan-locale-chunks skip condition is present with a Python check

Resolves #139


aidevops.sh v3.20.88 plugin for OpenCode v1.17.7 with gpt-5.5 spent 1m and 47,842 tokens on this as a headless worker.

Summary by CodeRabbit

  • Chores
    • Optimized documentation deployment workflow to skip unnecessary processing on scheduled runs when no changes are detected.

@superdav42 superdav42 added the origin:worker Created by headless/pulse worker session label Jun 18, 2026
@superdav42

Copy link
Copy Markdown
Contributor Author

Completion Summary

  • What: Added the same scheduled-run skip guard to the plan-locale-chunks workflow job that prepare-docs-source and build-locale-chunk already use, so unchanged scheduled runs do not spend CI minutes planning locale chunks.
  • Issue: Review followup: PR #137 — ci: parallelize locale deploy builds #139
  • Files changed: .github/workflows/deploy-docs.yml
  • Testing: actionlint unavailable; verified the expected plan-locale-chunks skip condition is present with a Python check
  • Key decisions: none

aidevops.sh v3.20.88 plugin for OpenCode v1.17.7 with gpt-5.5 spent 1m and 47,842 tokens on this as a headless worker.

@superdav42 superdav42 added the status:in-review PR open, awaiting review/merge label Jun 18, 2026
@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ccc1dadb-5ac8-46ec-94ba-5cd6050aca5f

📥 Commits

Reviewing files that changed from the base of the PR and between d0c7a20 and bd48e85.

📒 Files selected for processing (1)
  • .github/workflows/deploy-docs.yml

📝 Walkthrough

Walkthrough

Adds a job-level if: condition to the plan-locale-chunks job in .github/workflows/deploy-docs.yml, matching the same guard already present on other downstream jobs. The condition skips execution on scheduled runs when check-changes reports no documentation or addon changes.

Changes

Deploy Docs Workflow – Scheduled Run Guard

Layer / File(s) Summary
Skip condition on plan-locale-chunks
.github/workflows/deploy-docs.yml
Adds if: needs.check-changes.outputs.changed == 'true' || github.event_name != 'schedule' to the plan-locale-chunks job, aligning it with the skip logic already applied to prepare-docs-source and build-locale-chunk.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

review-feedback-scanned

Poem

🐇 A job ran on schedule without need,
Burning CI minutes like a weed.
One if: condition, two lines of care,
Now the bunny only hops when changes are there!
No more wasted runs — the warren stays lean. 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding a conditional skip to the plan-locale-chunks job to prevent execution during scheduled runs without changes.
Linked Issues check ✅ Passed The PR directly implements the specific requirement from issue #139: adding the conditional if statement to plan-locale-chunks at line 119 as requested by the review bot.
Out of Scope Changes check ✅ Passed The PR contains only the minimal, targeted change to .github/workflows/deploy-docs.yml required by issue #139, with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/auto-20260617-234516-gh139

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@superdav42 superdav42 merged commit a2fdced into main Jun 18, 2026
4 of 5 checks passed
@superdav42 superdav42 added the review-feedback-scanned Merged PR already scanned for quality feedback label Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker Created by headless/pulse worker session review-feedback-scanned Merged PR already scanned for quality feedback status:in-review PR open, awaiting review/merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review followup: PR #137 — ci: parallelize locale deploy builds

1 participant