-
-
Notifications
You must be signed in to change notification settings - Fork 495
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
[Documentation]: WordPress.DB.DirectDatabaseQuery #2458
base: develop
Are you sure you want to change the base?
[Documentation]: WordPress.DB.DirectDatabaseQuery #2458
Conversation
@jaymcp I noticed this PR is still in draft. Do you want me to review already ? Or are you still working on it ? |
@jaymcp No worries, take your time and thanks for letting me know. |
@jaymcp, I was just wondering if you'll have a chance to finish this off in the near future. It would be great if this PR could be included in the next WPCS release. If you haven't got time or lost interest, please let us know and we'll see if we can find someone to take over. Thanks! |
Hey @rodrigoprimo. I haven't lost interest, but I have unfortunately not regained my capacity. Sadly my optimism about being able to return to work on this turned out to be unfounded. My apologies. |
Hi @jaymcp Sorry to hear that. Hope things will start looking up again soon. Would you like someone else to take over the PR or would you prefer we leave it open for you to finish when you can ? |
Thank you, @jrfnl, and sorry for the slow reply. As much as I'd like to be able to finish what I started, if these docs are to go into the next release, it would probably best to unassign me. |
@jaymcp Thanks for letting us know. We'll keep an eye on the PR, but there's no hard release timeline, so we'll see if and when someone would be available to finish it off. |
Description
This PR adds documentation for the
WordPress.DB.DirectDatabaseQuery
sniff.Related issues/external references
Part of #1722