search

kourge / kvikk-fix

Apply prettier to all TypeScript files in your project
MIT License
1 stars 0 forks source link

--list-differences output is not helpful for newcomers #1

Open ianwremmel opened 6 years ago

ianwremmel commented 6 years ago

I struggled for a while to find a lint failure because a compound npm run script printed nothing but the name of a file:

$ npm run lint
running tslint && kvikk-fix -l
filename.ts

Maybe this could be changed to something like

$ npm run lint
running tslint && kvikk-fix --check

The following files are written in a way that violates your prettier rules

  - filename.ts

You can autofix them by running 'kvikk-fix'.

This change wouldn't remove --list-differences but augment it to have a human-friendly version as well as a machine-friendly one.

kourge commented 6 years ago

This is a good idea. The current behavior for --list-different mirrors that of prettier's CLI tool, which is very bare-bones and isn't user-friendly.