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.

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

STRWEB-110 ignore order of CSS imports #141

Merged
merged 2 commits into from
Apr 1, 2024
Merged

STRWEB-110 ignore order of CSS imports #141

merged 2 commits into from
Apr 1, 2024

Conversation

zburke
Copy link
Member

@zburke zburke commented Mar 14, 2024

Add @folio/consortia-settings to a platform build generates tens of thousands of lines of warnings form mini-css-extract-plugin about its inability to determine what order to assemble its CSS rules in because different files have the imports in a different order.

It seems likely we don't have to worry about this since per-module CSS will be scoped to that module by postcss, and shared CSS will just be in stripes-components, but this could be riskier than I realize, and a better approach may be to try to figure out what is happening in @folio/consortia-settings that is different than every other module we've added to the platform so far.

Refs STRWEB-110

Add `@folio/consortia-settings` to a platform build generates tens of
thousands of lines of warnings form mini-css-extract-plugin about its
inability to determine what order to assemble its CSS rules in because
different files have the imports in a different order.

It seems likely we don't have to worry about this since per-module CSS
will be scoped to that module by postcss, and shared CSS will just be in
stripes-components, but this could be riskier than I realize, and a
better approach may be to try to figure out what is happening in
`@folio/consortia-settings` that is different than every other module
we've added to the platform so far.

Refs STRWEB-110
@zburke zburke requested review from mkuklis and JohnC-80 March 14, 2024 23:39
Copy link

github-actions bot commented Mar 14, 2024

Jest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 915b44f. ± Comparison against base commit 3a4f947.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Mar 14, 2024

BigTest Unit Test Statistics

73 tests  ±0   73 ✔️ ±0   0s ⏱️ ±0s
39 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 915b44f. ± Comparison against base commit 3a4f947.

♻️ This comment has been updated with latest results.

@zburke zburke merged commit 93362cb into master Apr 1, 2024
5 checks passed
@zburke zburke deleted the STRWEB-110 branch April 1, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants