Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

[v8.x] Docs: Correct the order of loaders in postcss-loader example #1077

Merged
merged 1 commit into from
Sep 6, 2018
Merged

[v8.x] Docs: Correct the order of loaders in postcss-loader example #1077

merged 1 commit into from
Sep 6, 2018

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Sep 6, 2018

Since postcss-loader must be applied after sass-loader (and before css-loader) according to:
https://github.com/postcss/postcss-loader#config-cascade

...whereas previously it was being applied before sass-loader, since webpack applies loaders in the order of "last to first".

See:
#1066 (comment)

This is the release/v8 backport of #1076.

Since `postcss-loader` must be applied *after* `sass-loader` (and
before `css-loader`) according to:
https://github.com/postcss/postcss-loader#config-cascade

...whereas previously it was being applied *before* `sass-loader`,
since webpack applies loaders in the order of "last to first".

See:
#1066 (comment)

This is the `release/v8` backport of #1076.
@edmorley edmorley added the docs label Sep 6, 2018
@edmorley edmorley self-assigned this Sep 6, 2018
@edmorley
Copy link
Member Author

edmorley commented Sep 6, 2018

CI is failing due to an unrelated issue, for which I've filed #1078.
Seeing that this is docs-only and the Netlify build passed, I'm just going to merge this anyway.

@edmorley edmorley merged commit 9542cb1 into neutrinojs:release/v8 Sep 6, 2018
@edmorley edmorley deleted the v8-fix-postcss-docs branch September 6, 2018 15:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

1 participant