search

mariocasciaro / gulp-concat-css

Concatenates css files, bubbling up import statements (as per the standard), and optionally rebasing urls and inlining local import statements.
MIT License
78 stars 19 forks source link

Error while parsing @import #13

Closed nsotnikov closed 9 years ago

nsotnikov commented 9 years ago

Hello, have an error by integrating gulp-concat-css in my gulp.js

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: Could not find a valid import path in string: @import url(http://fonts.googleapis.com/css?family=Open+Sans)

OK, I found that semicolon is a problem. Take a look: 

@import "//netdna.bootstrapcdn.com/font-awesome/4.2.0/css/font-awesome.css" // working
@import "//netdna.bootstrapcdn.com/font-awesome/4.2.0/css/font-awesome.css"; // NOT working

My configuration:

mariocasciaro commented 9 years ago

Confirmed that the problem is caused by a breaking change in https://github.com/kevva/import-regex

mariocasciaro commented 9 years ago

Fix landed in 1.1.1

nsotnikov commented 9 years ago

Yes, it works very well. Thanks for bugfix.