Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Wrong import precedence #557

Merged
merged 1 commit into from
Apr 13, 2018
Merged

fix: Wrong import precedence #557

merged 1 commit into from
Apr 13, 2018

Conversation

jhnns
Copy link
Member

@jhnns jhnns commented Apr 13, 2018

- Fixes problem with foundation
- Fixes #556
@codecov
Copy link

codecov bot commented Apr 13, 2018

Codecov Report

Merging #557 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #557   +/-   ##
=======================================
  Coverage   97.58%   97.58%           
=======================================
  Files           6        6           
  Lines         124      124           
=======================================
  Hits          121      121           
  Misses          3        3
Impacted Files Coverage Δ
lib/importsToResolve.js 100% <100%> (ø) ⬆️

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 9409e17...03e5f3e. Read the comment docs.

@jhnns jhnns merged commit f4eeff1 into master Apr 13, 2018
@jhnns jhnns deleted the fix/imports-556 branch April 13, 2018 14:06
@@ -15,3 +15,10 @@
@import ~animate.css/animate
/* @import url(http://example.com/something/from/the/interwebs); */
@import url(http://example.com/something/from/the/interwebs);
/* scoped import @import language */
.scoped-imporr
Copy link

Choose a reason for hiding this comment

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

imporr typo?

Copy link
Member

Choose a reason for hiding this comment

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

@Yiin just class name

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants