-
Notifications
You must be signed in to change notification settings - Fork 60
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
Add wp db clean
command
#93
Conversation
Will take a look at the errors later today. |
@marksabbath Are you still up for working on this command? Quick note about the PR: An unrelated test now fails because you changed surrounding whitespace that is unrelated to your code change. Please make sure your editor does not introduce unwanted changes, to keep the PR clean and targeted. |
Yes @schlessera working on it BTW. Planning to finish that up by today. |
Hey, @schlessera it seems that 2 errors not related to my code are blocking this merge. 1 - WordPress 5.0 seems to have changed the tables on Multisite installs (see https://travis-ci.org/wp-cli/db-command/jobs/362222469); Should I update the tests of |
@marksabbath Both issues should be resolved. If you merge latest |
Done @schlessera all good now! Thanks man! |
Anything else we would need to do to move this forward @schlessera ? |
Thanks for the pull request, @marksabbath, and sorry for the wait. I did some minor changes, and this is good to be merged now. |
This should resolve #92 |
Hi, this command is not removing all tables with $table_prefix: custom tables are not removed. If we have Did I miss something or is it a bug? ps. I have read that db clear was made to "limit" db reset: I'm using wp db to link git commit to db status and clean command is the best option I found.
|
Add `wp db clean` command
wp db clean
Introducing new
wp db clean
. This new command drops only the tables that are present in$table_prefix
.