Skip to content

docs(tests): collapse compounded footer blocks - #3447

Merged
eleshar merged 1 commit into
developfrom
fix/tests-readme-footer-restore
Sep 22, 2026
Merged

eleshar merged 1 commit into
developfrom
fix/tests-readme-footer-restore

Conversation

@eleshar

@eleshar eleshar commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

linked issues

N/A - residue cleanup from automation path proving (PR #3440) plus pre-existing compounding.

context

  • Severity/Impact: Low (docs cosmetics, 72 junk lines)
  • Affected versions/environments: tests/README.md

reproduction

root cause

fix summary

  • Replaced the 25-block tail with the single canonical footer (the exact form meta.agent.js writes for this file).

verification

  • Ran meta.agent.js against the cleaned file: changed:false (idempotent, no further normalization)
  • Line count 428 to 356, single footer block remaining

risk & rollback

  • Risk level: Low
  • Rollback plan: revert this commit

changelog

Fixed

  • Compounded footers cleanup: collapsed 25 duplicated footer blocks in tests/README.md to the single canonical footer.

Checklist (Global DoD / PR)

  • All AC met and demonstrated
  • Tests added/updated (unit/E2E as appropriate)
  • Accessibility checklist completed (where relevant): N/A - footer dedup, no UI
  • Docs/readme/changelog updated (if user-facing)
  • Security checklist completed (where relevant): N/A - docs only, no input or secrets
  • Code/design reviews approved
  • CI green; linked issues closed; release notes prepared (if shipping)

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 56 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: cd81fd6d-da5d-4031-9c63-00abf2d67e7d

📥 Commits

Reviewing files that changed from the base of the PR and between 642cced and 3f33247.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • tests/README.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

📋 Changelog Quality Validation

Summary

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

Status

❌ Validation FAILED - This PR introduces 1 new changelog failure(s). Please fix them before merging.
Pre-existing failures (7) 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 3447 --branch fix/tests-readme-footer-restore --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

Copy link
Copy Markdown
Contributor

PR Template Routing

Branch Type: fix
Scope: tests-readme-footer-restore
Template: pr_bug.md
Labels Applied: type:bug,area:integration,area:documentation,area:testing

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

@mergify

mergify Bot commented Sep 22, 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

@eleshar
eleshar merged commit e1f3d35 into develop Sep 22, 2026
23 checks passed
@eleshar
eleshar deleted the fix/tests-readme-footer-restore branch September 22, 2026 11:27
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