Skip to content

Move postcss to devDependencies in "css-syntax-patches-for-csstree"#1708

Merged
romainmenke merged 1 commit intocsstools:mainfrom
jnoordsij:fix-postcss-peerdep
Oct 30, 2025
Merged

Move postcss to devDependencies in "css-syntax-patches-for-csstree"#1708
romainmenke merged 1 commit intocsstools:mainfrom
jnoordsij:fix-postcss-peerdep

Conversation

@jnoordsij
Copy link
Copy Markdown
Contributor

When installing this package as nested dependency of cssstyle with Yarn PnP on the latest Yarn version (4.10.3), one is presented with the following error:

➤ YN0086: │ Some peer dependencies are incorrectly met by dependencies; run yarn explain peer-requirements for details.

which when inspected can be traced to

pb97ead → ✘ cssstyle@npm:5.3.1 doesn't provide postcss to @csstools/css-syntax-patches-for-csstree@npm:1.0.14 [bb38a]

Given that seemingly this package does not interact with postcss when inspecting the code, apart from in a single e2e test, I think moving it to be a dev dependency is more accurate.

Copy link
Copy Markdown
Member

@romainmenke romainmenke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! thank you 🙇

@romainmenke romainmenke merged commit 0afa6c5 into csstools:main Oct 30, 2025
8 checks passed
@romainmenke
Copy link
Copy Markdown
Member

This was released, thank you again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants