Skip to content

Remove WP 4.2 requirement note#626

Merged
swissspidy merged 1 commit into
mainfrom
fix/comment
Jul 23, 2026
Merged

Remove WP 4.2 requirement note#626
swissspidy merged 1 commit into
mainfrom
fix/comment

Conversation

@swissspidy

@swissspidy swissspidy commented Jul 23, 2026

Copy link
Copy Markdown
Member

We require WP 4.9+ nowadays, so this is obsolete

Summary by CodeRabbit

  • Documentation
    • Updated the --autoload option description for the update command by removing an outdated WordPress version requirement note.

We require WP 4.9+ nowadays, so this is obsolete
@swissspidy
swissspidy requested a review from a team as a code owner July 23, 2026 12:30
@swissspidy swissspidy added the scope:documentation Related to documentation label Jul 23, 2026
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 192c857e-68b7-430d-8ef5-f378434d307f

📥 Commits

Reviewing files that changed from the base of the PR and between 3d5d5a9 and dff0a75.

📒 Files selected for processing (1)
  • src/Option_Command.php

📝 Walkthrough

Walkthrough

The --autoload documentation for the update command removes the “Requires WP 4.2.” note while leaving the rest of the option description unchanged.

Changes

Option documentation

Layer / File(s) Summary
Update command option description
src/Option_Command.php
The --autoload parameter description no longer states that it requires WordPress 4.2.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 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 describes the main change: removing the obsolete WordPress 4.2 requirement note.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/comment

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

@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@swissspidy swissspidy added this to the 2.8.13 milestone Jul 23, 2026
@swissspidy
swissspidy merged commit b49c339 into main Jul 23, 2026
78 checks passed
@swissspidy
swissspidy deleted the fix/comment branch July 23, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:documentation Related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant