search

duncanjbrown / WP-CLI-Migrate

A database migration command for WP-CLI
32 stars 3 forks source link

Fixed function documentation to new wp-cli standards #3

Closed MartijnBraam closed 10 years ago

MartijnBraam commented 10 years ago

I got this failure when running:

$ wp migrate to /srv/http/test.dev http://test.dev/ test.sql   
Warning: The `wp migrate to` command has an invalid synopsis part: output_folder
Warning: The `wp migrate to` command has an invalid synopsis part: output_host
Warning: The `wp migrate to` command has an invalid synopsis part: filename
Error: Too many positional arguments: /srv/http/test.dev http://test.dev/ test.sql

Now it works

duncanjbrown commented 10 years ago

Hi, thanks for the patch! On testing this I found I still got the synopsis complaint for output_folder and output_host. Removing the underscores from these seemed to fix it - perhaps that's a Markdown issue? Could you possibly take them out? Very happy to merge after that.

clemens-tolboom commented 10 years ago

I just tested the PR and got no errors

$ wp migrate to
usage: wp migrate to <output_folder> <output_host> <filename>

So just curious what version of wp-cli you guys use. I use wp-cli git clone

duncanjbrown commented 10 years ago

@clemens-tolboom Yes it works with the latest WP-CLI, thanks for pointing that out. Merging this. Thanks @MartijnBraam.