Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve config documentation and remove 22 dependencies from cssnano #1168

Merged
merged 2 commits into from Jul 19, 2021
Merged

Improve config documentation and remove 22 dependencies from cssnano #1168

merged 2 commits into from Jul 19, 2021

Commits on Jul 18, 2021

  1. test: add configuration loading tests and docs 

    Closes #904
Add tests for configuration loading and correct the documentation.

* correct configuration file names in the docs
* specify that the PostCss config overrides the cssnano-specific config
* remove mention of cosmiconfig to avoid being tied to a single implementation
    @ludofischer
    ludofischer committed Jul 18, 2021
    Copy the full SHA
    786969b View commit details
    Browse the repository at this point in the history

  2. chore(deps): replace cosmiconfig with lilconfig 

    Remove about 22 dependencies according to npm i --production (from 110 to 88 deps)

* reduce node_modules after `yarn install cssnano` in empty project from 14MB to 13MB
* add YAML dep to preserve YAML configuration compatibility
  YAML configuration was only implicitly documented by referring to cosmiconfig,
  but removing it would be a breaking change
    @ludofischer
    ludofischer committed Jul 18, 2021
    Copy the full SHA
    8e39090 View commit details
    Browse the repository at this point in the history