postcss in install instructions

[receter]

Is postcss still a dependency? Its's no in the peerDependencies, but it is in the install script in the README.

I am asking because I am not sure if I really need to install it. It works without explicitly installing it for me, but I use Vite so postcss is installed because of Vite.

Thank you, I really appreciate you published this plugin!

[Siilwyn]

PostCSS is listed in the peer dependencies:

prettier-plugin-css-order/package.json

Line 23 in 0948dc8

"prettier": "3.x"

Or do you mean something else?

[receter]

Hmm, then it's a issue with my eyes. Sorry for the inconvenience.

[receter]

This is prettier right?

[receter]

#46

[Siilwyn]

Oh whoops sorry about that. Regarding postcss, I never call postcss directly from this codebase and it's a peer dependency of css-declaration-sorter (which is a dependency here). Peer dependencies are automatically installed with npm 7>, if it isn't already in the parent tree.

So yes it can be removed from the readme instructions I think, this is the tree I get with just installing this package:

└─┬ prettier-plugin-css-order@2.1.2
  ├─┬ css-declaration-sorter@7.2.0
  │ └─┬ postcss@8.4.47
  │   ├── nanoid@3.3.7
  │   ├── picocolors@1.1.1
  │   └── source-map-js@1.2.1
  ├─┬ postcss-less@6.0.0
  │ └── postcss@8.4.47 deduped
  ├─┬ postcss-scss@4.0.9
  │ └── postcss@8.4.47 deduped
  └── prettier@3.3.3

[receter]

Ok, this makes sense. It didn't work for me because I accidentally had legacy-peer-deps=true in my global .npmrc, glad I found this… must have been there a while already 😬

[Siilwyn]

Ah good find! Thank you for the thoughts on this I've removed postcss from the installation instruction too since it might be confusing and isn't needed in most cases.