Skip to content

fix: prerelease cleanup - #358

Merged
justlevine merged 2 commits into
mainfrom
fix/release-cleanup
Oct 21, 2026
Merged

justlevine merged 2 commits into
mainfrom
fix/release-cleanup

Conversation

@justlevine

@justlevine justlevine commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

What

  • Casts a value to int to prevent escaping warnings from Plugin Check
  • Drops two dead links from DEVELOPMENT.md's TOC

Why

Prerelease cleanup

Related Issue(s):

How

AI Disclosure

Testing Instructions

Screenshots

Additional Info

Checklist

  • I have read the Contribution Guidelines.
  • I have read the Development Guidelines.
  • I have added necessary tests to cover my changes.
  • I have updated the project documentation as needed.
  • My code has detailed inline documentation.
  • My code is tested to the best of my abilities.
  • My code passes all lints, tests, and checks.
Open WordPress Playground Preview

Copilot AI balanced review requested due to automatic review settings October 21, 2026 12:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

Both changes are narrowly scoped, preserve existing behavior, and have no identified issues.

No open issues 🎉

What changed in this PR

Small prerelease cleanup addressing a Plugin Check warning and outdated documentation links.

Changes:

  • Explicitly casts the Algolia task ID to int in an exception message.
  • Removes two dead links from the development guide’s table of contents.
File Description
inc/​Algolia/​Index.php Adds an integer cast when formatting the task ID.
docs/​DEVELOPMENT.md Removes obsolete test documentation links.

Review effort: Balanced

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.05%. Comparing base (014ff0a) to head (c649b3f).

Files with missing lines Patch % Lines
inc/Algolia/Index.php 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #358      +/-   ##
============================================
+ Coverage     87.01%   88.05%   +1.04%     
  Complexity      677      677              
============================================
  Files            36       26      -10     
  Lines          2742     2169     -573     
  Branches        170        0     -170     
============================================
- Hits           2386     1910     -476     
+ Misses          297      259      -38     
+ Partials         59        0      -59     
Flag Coverage Δ
unit 88.05% <0.00%> (ø)
vitest ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
inc/Algolia/Index.php 96.92% <0.00%> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@justlevine
justlevine merged commit fdf8d46 into main Oct 21, 2026
19 checks passed
@justlevine
justlevine deleted the fix/release-cleanup branch October 21, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants