search

10up / MU-Migration

This WP-CLI plugin makes the process of moving sites from single WordPress sites to a Multisite instance (or vice-versa) much easier. It exports everything into a zip package which can be used to automatically import it within the desired Multisite installation.
MIT License
329 stars 47 forks source link

Improve docblocks #22

Closed noplanman closed 7 years ago

noplanman commented 7 years ago

Complete most docblocks, setting the proper parameters and return statements, including their data types.

I haven't added a description for ALL parameters yet, which might not even be necessary?

nicholasio commented 7 years ago

Looks great! Is this ready for merge?

noplanman commented 7 years ago

Kind of. There isn't a great description for all parameters and some of them are missing.

I'm not sure if it's necessary to put them all or let the parameter name be self-explanatory.

(Will comment on a few places inline)

noplanman commented 7 years ago

Right, have sieved through again and fixed most remaining docblocks and inline comments.

@nicholasio You can have another look and then merge if you're happy.

nicholasio commented 7 years ago

@noplanman Looks good. Please fix conflicts and I'll merge this.

noplanman commented 7 years ago

Ok, all clear 👌