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

Leading comments break compiling #13

Open
deniaz opened this issue Dec 2, 2016 · 2 comments
Open

Leading comments break compiling #13

deniaz opened this issue Dec 2, 2016 · 2 comments

Comments

@deniaz
Copy link

deniaz commented Dec 2, 2016

The following SCSS file

/**
 * Test
 */

$foo: 'bar';

leads to the following error:

<path>/node_modules/node-sass/lib/index.js:440
  throw assign(new Error(), JSON.parse(result.error));
  ^

Error: Invalid CSS after ".test { content:": expected expression (e.g. 1px, bold), was "/** };"
    at Object.module.exports.renderSync (<path>/node_modules/node-sass/lib/index.js:440:16)
    at Object.fromString (<path>/node_modules/scss-to-json/src/compile.js:17:18)
    at Value._parse (<path>/node_modules/scss-to-json/src/value.js:17:28)
    at Value (<path>/node_modules/scss-to-json/src/value.js:11:8)
    at Declaration._parse (<path>/node_modules/scss-to-json/src/declaration.js:26:18)
    at Declaration (<path>/node_modules/scss-to-json/src/declaration.js:14:8)
    at <path>/node_modules/scss-to-json/src/processor.js:90:12
    at Array.map (native)
    at declarationsFromString (<path>/node_modules/scss-to-json/src/processor.js:89:16)
    at new Processor (<path>/node_modules/scss-to-json/src/processor.js:104:18)

Yet node-sass will compile it correctly. I might investigate further and create a PR if I find something.

@Saturate
Copy link

Saturate commented Nov 6, 2017

I can confirm this issue, it's breaking on all of my files.

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

No branches or pull requests

3 participants