search

clean-css / clean-css-cli

The command line interface to clean-css CSS optimizer
MIT License
302 stars 17 forks source link

Update readme to replace `--level` option with `-O` #68

Closed davidjb closed 3 years ago

davidjb commented 3 years ago

This change removes mention of the --level option which never existed. In versions earlier than 5.0.0, clean-css-cli used to accept the invalid option silently. With versions > 5.0.0, trying to use the option results in the following: error: unknown option '--level'. The root cause of the change the upgrade to commander v7.0.0, where unknown (excess) command arguments are now raised as an error.

For interest, I was using the option because Bootstrap previously used it in their package.json.

jakubpawlowicz commented 3 years ago

Hi @davidjb - many thanks for spotting this! You are right, there were never a --level option in the CLI. Merging right away.

davidjb commented 3 years ago

No problem @jakubpawlowicz - I had to go digging to understand what happened on upgrading from v4 to v5..and turns out it was never there at all :man_shrugging: