Code review comment for lp:~jml/subunit/to-csv

Revision history for this message
Jonathan Lange (jml) wrote :

I've factored out the common code from subunit2csv, subunit2junitxml and subunit-notify. This should give us a good basis for factoring out the rest of the scripts, which all have slightly different options (I want to avoid making any behavioural changes in this branch other than adding subunit2csv).

As for --no-passthrough being the default, the same argument applies to subunit2junitxml, and now even the code is the same. I would like to avoid tackling that problem in this branch, since it spreads behavioural change to other scripts (unless I defactor a little), and since it would involve dealing with backwards compatibility issues, which are always tricky.

« Back to merge proposal