search

thaljef / Pinto

Curate your own repository of Perl modules
https://metacpan.org/module/Pinto::Manual
66 stars 49 forks source link

Add custom diff formats #200

Closed rafl closed 9 years ago

rafl commented 9 years ago

This extends the existing '--format' option already available for the 'list' command to 'diff', while preserving compatibility with the existing '--diff-style' option.

thaljef commented 9 years ago

Maybe this is just a matter of rewording the documentation, but I think the --format and the --diff-style are orthogonal. The style controls whether the output records will be distributions or packages. So the format would control how each of those records will be presented.

rafl commented 9 years ago

That's a very good point. I had missed the fact that --diff-style controls more than just the format used to print each individual line. I've updated this pull request accordingly.

thaljef commented 9 years ago

That looks good. The existing presentation logic for diffs is a bit scattered, so I'd like to consolidate that too. I'll try to merge and release this over the weekend.