Ryan McCue
Ryan McCue
@drzraf Do not spam comment this repository.
(See comment on #92.)
Related to #53 I think.
To fit better with how CLI apps generally work, you should pass `-` instead, and we should handle that if we don't already.
> And when someone already is taking on this, the following 100+ lines in the switch could drastically be simplified: FWIW, the performance degradation here was significant last time I...
Haven't had a chance to review the code again yet, just wanted to note that there :) I'll try and take a look in the morning.
Parsing the schema sounds potentially expensive, as it'd require a full runthrough of the file for validation. We should be able to pick this up during the preliminary stage, I'd...
The UI already does a preliminary parsing stage for this, so that'd require 3 runthroughs of the file. I think that's a little too expensive, but willing to be proven...
Repurposing this ticket ;)
> state clearly it replaces the WordPress Importer and uses the same CLI arguments. (ie. deactivate the old one) Oh no, I forgot to document this! You actually want `wp...