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

fix: add dynamic imports to the end of the css file #317

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wardpeet
Copy link
Collaborator

#295 did not take care of dynamicImported modules. These modules were included but at the top of the css file. We can't know the execution sequence of these imports but I've added them to the recursive function so they are ranked lower than regular imports.

Fixes #316

@codecov
Copy link

codecov bot commented Aug 30, 2020

Codecov Report

Merging #317 (b392bd7) into master (6480f02) will decrease coverage by 0.19%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
- Coverage   92.94%   92.74%   -0.20%     
==========================================
  Files          10       10              
  Lines         326      331       +5     
  Branches      115      115              
==========================================
+ Hits          303      307       +4     
- Misses         22       23       +1     
  Partials        1        1              
Impacted Files Coverage Δ
src/index.js 99.08% <100.00%> (-0.92%) ⬇️
src/less-loader.js 83.33% <0.00%> (ø)
src/sass-loader.js 88.88% <0.00%> (ø)
src/stylus-loader.js 81.81% <0.00%> (ø)
src/postcss-loader.js 92.00% <0.00%> (ø)
src/utils/normalize-path.js 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6480f02...47e91c0. Read the comment docs.

@rproserpio
Copy link

@egoist any chance @wardpeet PR can reviewed and merged? It's a fix for a regression introduced in the last release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants