Skip to content

Improve & document WP-CLI hook usage - #160

Merged
swissspidy merged 1 commit into
mainfrom
fix/do-hook
Jul 30, 2026
Merged

Improve & document WP-CLI hook usage#160
swissspidy merged 1 commit into
mainfrom
fix/do-hook

Conversation

@swissspidy

@swissspidy swissspidy commented Jul 30, 2026

Copy link
Copy Markdown
Member

Relies on wp-cli/wp-cli-tests#344

Summary by CodeRabbit

  • Documentation
    • Expanded developer documentation for the REST request arguments hook, including callback parameters.
    • Clarified how request arguments can be customized without changing request behavior.

@swissspidy swissspidy added this to the 0.4.3 milestone Jul 30, 2026
@swissspidy
swissspidy requested a review from a team as a code owner July 30, 2026 09:23
@github-actions github-actions Bot added dependencies Pull requests that update a dependency file scope:documentation Related to documentation scope:framework labels Jul 30, 2026
@coderabbitai

coderabbitai Bot commented Jul 30, 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: 7e8794a7-c3ce-43d3-90db-4718db879a1e

📥 Commits

Reviewing files that changed from the base of the PR and between d222ade and 730e3a5.

📒 Files selected for processing (1)
  • inc/RestCommand.php

📝 Walkthrough

Walkthrough

The RestCommand PHPDoc now documents the restful_http_request_args hook, its request argument structure, and the RestCommand callback context. Request-building logic remains unchanged.

Changes

REST request documentation

Layer / File(s) Summary
Document request arguments hook
inc/RestCommand.php
Expanded PHPDoc describes the restful_http_request_args hook and its $request_args and RestCommand $command callback parameters.

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

Suggested reviewers: brianhenryie

🚥 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 matches the change: it documents WP-CLI hook usage and improves hook-related docs.
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/do-hook

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.

@codecov

codecov Bot commented Jul 30, 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 merged commit 64458ef into main Jul 30, 2026
78 checks passed
@swissspidy
swissspidy deleted the fix/do-hook branch July 30, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file scope:documentation Related to documentation scope:framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant