-
Notifications
You must be signed in to change notification settings - Fork 998
Added PHPDoc for cmd_starts_with() method #6034
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I believe we have a ton of missing docblocks in a lot of other places—would you like to help improve them more holistically? :) |
Thank you for pointing that out! I'd be happy to help improve the docblocks more holistically. Let me know if there's a specific area you'd like me to focus on. 😊 |
Pretty much everything in the |
Actually, many methods are missing docblocks, and I’m not entirely sure about the purpose of each method. How can I write a proper docblock for the methods in the PHP folder without fully understanding the functionality of each one? |
Fair enough :) let‘s start with this one then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I don't meant to be so Nitpicky on a small PR but this is a particular case where cmd / command happens to refer to the WP-CLI binary, while in most other contexts it actually refers to a specific wp cli command (user, core, db, etc...) so it is better to clarify (even in the next function below "command" starts to refer to specific wp-cli commands again
Thanks! |
Fix: #6033
Add PHPDoc for
cmd_starts_with()
method inRunner
class to improve code readability, align with WordPress coding standards, and help developers understand the purpose and behaviour of the function.