Skip to content

fix(ci): bot maintenance via PR, throttle metrics schedule to daily - #3410

Open
eleshar wants to merge 2 commits into
developfrom
ci/bot-pr-noise-reduction
Open

eleshar wants to merge 2 commits into
developfrom
ci/bot-pr-noise-reduction

Conversation

@eleshar

@eleshar eleshar commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Linked issues

N/A — follow-up to #3407, same Ash request to stop bot direct-pushes and reduce noise.

Build/CI change

  • What: docs maintenance job (manual dispatch) now opens/updates fixed-branch PR chore/docs-maintenance via peter-evans/create-pull-request instead of pushing to the checked-out branch. Metrics aggregator schedule hourly -> daily 03:17 UTC backstop (workflow_run trigger keeps the PR fresh on activity).
  • Why: last direct-push path removed; hourly schedule burned minutes and PR-update noise for a backstop job.

Baseline & Target

  • Before: 1 direct-push path left (maintain job); metrics ran 24x/day on schedule plus every validation completion.
  • After: zero direct pushes in all three bot jobs; metrics scheduled 1x/day off-hour, event-driven otherwise. Single updating PR per bot (no PR-per-run).

Rollback

Plan: revert this PR; maintain job returns to direct push, metrics to hourly schedule.

Notes

  • Secrets/permissions: create-pull-request uses default GITHUB_TOKEN; its PRs do not retrigger push/PR CI by design (per upstream docs). PAT/App token can be added later if CI on bot PRs is required.
  • Noise approach per GitHub 2026 guidance (grouped single PR, slow cadence, off-hour schedule, silent no-op when no diff).

Changelog

Fixed

  • Documentation maintenance no longer pushes directly; metrics schedule throttled to daily.

@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: lightspeedwp/.github/.coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: 8d588999-25c5-4ceb-a4bf-214547688c04

📥 Commits

Reviewing files that changed from the base of the PR and between d1424a4 and efbc4d1.

📒 Files selected for processing (3)
  • .github/workflows/branch-validation-metrics-aggregator.yml
  • .github/workflows/documentation.yml
  • CHANGELOG.md

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.

@github-actions

Copy link
Copy Markdown
Contributor

PR Template Routing

Branch Type: ci
Scope: bot-pr-noise-reduction
Template: pr_ci.md
Labels Applied: type:ci,area:ci

This PR was automatically routed based on the branch naming strategy.

@mergify

mergify Bot commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@github-actions

Copy link
Copy Markdown
Contributor

📋 Changelog Quality Validation

Summary

Metric Count
✅ Passing 73
⚠️ Warnings 0
❌ Failing 1
🆕 New failures in this PR 1
📦 Pre-existing failures 0

Status

Validation FAILED - This PR introduces 1 new changelog failure(s). Please fix them before merging.
Pre-existing failures (0) are reported for visibility and do not block this PR.

How to Fix

Reproduce the exact PR check locally (from the repository root):

cd .github/validation/changelog
node bin/validate.js --changelog-path ../../../CHANGELOG.md --trigger pr_submission --pr-number 3410 --branch ci/bot-pr-noise-reduction --output text

Only new failures introduced by this PR block merging; pre-existing Unreleased failures do not block. See Changelog Quality Audit for the rule reference.

@github-actions github-actions Bot added area:ci and removed area:ci labels Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant