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

Error when linting CSS string with cache enabled. #2492

Closed
olegskl opened this issue Apr 11, 2017 · 3 comments
Closed

Error when linting CSS string with cache enabled. #2492

olegskl opened this issue Apr 11, 2017 · 3 comments
Assignees
Labels
type: bug a problem with a feature or rule

Comments

@olegskl
Copy link

olegskl commented Apr 11, 2017

Describe the issue. Is it a bug or a feature request (new rule, new option, etc.)?

An error is thrown when trying to lint a CSS string while cache option is set to true.

Which rule, if any, is this issue related to?

Not applicable.

What CSS is needed to reproduce this issue?

Any CSS string.

What stylelint configuration is needed to reproduce this issue?

{
  "code": ".foo {}",
  "codeFilename": "foo.css",
  "cache": true
}

Which version of stylelint are you using?

7.10.1

How are you running stylelint: CLI, PostCSS plugin, Node API?

const stylelint = require('stylelint');

stylelint.lint({
  code: '.foo {}',
  codeFilename: 'foo.css',
  cache: true,
  config: {rules: []}
}).catch(console.error);

Does your issue relate to non-standard syntax (e.g. SCSS, nesting, etc.)?

No.

What did you expect to happen?

Lint should work. Not sure about caching (probably should be ignored?).

What actually happened (e.g. what warnings or errors you are getting)?

TypeError: Cannot read property 'reconcile' of undefined
    at prepareReturnValue (...\node_modules\stylelint\lib\standalone.js:174:16)
    at stylelint._lintSource.then.catch.then.stylelintResult (...\node_modules\stylelint\
lib\standalone.js:90:14)
@alexander-akait alexander-akait added status: ready to implement is ready to be worked on by someone type: bug a problem with a feature or rule labels Apr 11, 2017
@alexander-akait
Copy link
Member

@olegskl thanks for issue 👍

@sergesemashko
Copy link
Contributor

thanks @olegskl, I'll take care of it.

@sergesemashko sergesemashko self-assigned this Apr 11, 2017
@jeddy3 jeddy3 added status: wip is being worked on by someone and removed status: ready to implement is ready to be worked on by someone labels Apr 11, 2017
sergesemashko pushed a commit to sergesemashko/stylelint that referenced this issue Apr 12, 2017
sergesemashko pushed a commit to sergesemashko/stylelint that referenced this issue Apr 12, 2017
@sergesemashko sergesemashko added PR: review needed status: wip is being worked on by someone and removed status: wip is being worked on by someone PR: review needed labels Apr 12, 2017
@davidtheclark
Copy link
Contributor

Closed by #2494.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug a problem with a feature or rule
Development

No branches or pull requests

6 participants