search

simonw / csv-diff

Python CLI tool and library for diffing CSV and JSON files
Apache License 2.0
292 stars 47 forks source link

Option to also print all fields in changed rows #9

Closed pbrende closed 3 years ago

pbrende commented 3 years ago

When visually scanning the output, it would be nice to also have the un-changed fields. Often, critical context is provided by those fields.

simonw commented 3 years ago

Good idea. I'll add a --show-unchanged option.

simonw commented 3 years ago

Documentation: https://github.com/simonw/csv-diff/commit/873fde4c838ea84fc26ae3820b98b79a80a5d742?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

pbrende commented 3 years ago

Awesome. Works great. I think there's a typo in the documentation. I believe it should be: ... to include full details of the changed rows...

On Thu, Oct 15, 2020 at 6:34 PM Simon Willison notifications@github.com wrote:

Documentation: 873fde4 ?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 https://github.com/simonw/csv-diff/commit/873fde4c838ea84fc26ae3820b98b79a80a5d742?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/simonw/csv-diff/issues/9#issuecomment-709677010, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALPQCVQ54ITRCRXAJ2XTDNDSK6PMDANCNFSM4SSUKQNA .

simonw commented 3 years ago

Yes good spot, thanks!