node-wp-cli icon indicating copy to clipboard operation
node-wp-cli copied to clipboard

Fixing issue #12 correct handling of format argument

Open craigedmunds opened this issue 8 years ago • 1 comments

I don't think the defaulting of format to json is necessary at all? Although the absence of tests makes it difficult to gauge!

craigedmunds avatar Jul 19 '16 10:07 craigedmunds

I agree, not sure what the rationale is behind those original lines of code (if the format parameter already exists, then change it to json, but then if it doesn't exist, don't do anything?). I had to use your fork in my project to make any progress in using the search_replace command, thanks for posting it.

farmerpaul avatar Mar 17 '21 18:03 farmerpaul