Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

docs: use postcss-loader's new postcssOptions #198

Merged
merged 3 commits into from Oct 27, 2021
Merged

docs: use postcss-loader's new postcssOptions #198

merged 3 commits into from Oct 27, 2021

Conversation

nicoqh
Copy link
Contributor

@nicoqh nicoqh commented Jan 31, 2021

PostCSS options were moved to postcssOptions.

PostCSS options were moved to the `postcssOptions` option.
INSTALL.md Outdated
loader: 'postcss-loader',
options: {
postcssOptions: {
ident: 'postcss',

Choose a reason for hiding this comment

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

That will fail.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated to use an array instead of a function. This is identical to my own setup:)

Choose a reason for hiding this comment

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

It's the ident property, sorry. It will crash, because it does not validate as a proper config key.

Copy link
Contributor Author

@nicoqh nicoqh Feb 1, 2021

Choose a reason for hiding this comment

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

Oh, I wasn't aware they had removed it (and I didn't get any warnings). Sorry for the trouble! (I can rebase if you want.)

@jonathantneal jonathantneal merged commit 9e59c39 into csstools:main Oct 27, 2021
romainmenke pushed a commit to csstools/postcss-plugins that referenced this pull request Nov 16, 2021
…t-env#198)

* docs: use the new postcssOptions

PostCSS options were moved to the `postcssOptions` option.

* use a plugin array in the webpack example

* remove the ident option from the webpack example
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants