search

postcss / postcss-cli

CLI for postcss
Other
836 stars 93 forks source link

refactor: replace chalk with colorette #365

Closed ludofischer closed 3 years ago

ludofischer commented 3 years ago

Since postcss and postcss-reporter use colorette, getting rid of chalk in this package saves about 100KB when doing a yarn install.

I’ve run a few commands (postcss, postcss --watch) to check the output matches visually. We could diff programmatically a string highlighted by chalk with one highlighted by colorette, but then we would just be testing whether chalk and/or colorette got the color names right, which I assume they did.

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 81.41% when pulling ba5b6efde053e8a0ee15b973d66410e4c353b940 on ludofischer:colorette into 35545bdb384d0e75ffa4385de960968f4a6cd28d on postcss:master.