Fixed: set errored to false if warning no contains errors.

[alexander-akait]

Closes #22

BTW, i would recommend don't changed stylelintResult, just add node in README.md about disable no-empty-source and no-missing-end-of-source-newline, in next version we can change stylelintResult (example rename warnings to another name) and stylelint-processor-styled-components will cease to work as expected.

[mxstbr]

Thanks!

[mxstbr]

in next version we can change stylelintResult (example rename warnings to another name) and stylelint-processor-styled-components will cease to work as expected.

That would be a major version release anyway, in which case we'd have to update and release a major version too. Might as well make that explicit rather than only mentioning it in the docs and avoid a footgun for people?

[mxstbr]

Can you fix the linting errors?

   98:1  error  Trailing spaces not allowed                                     no-trailing-spaces
  100:7  error  Assignment to property of function parameter 'stylelintResult'  no-param-reassign

[ghost]

BTW, i would recommend don't changed stylelintResult

@evilebottnawi Next to this pr as a patch, I think it would be a good solution to define the rules that need to be disabled in a shareable config. Users can then extend that config and list it before all others so it overrides all other rules (https://stylelint.io/user-guide/configuration/#extends).

Say for example: stylelint-config-styled-components

Then we could leave stylelintResult as is. I think defining the shareable config as a peerDependency could be a good way to allow coupling a version of this lib to a version of the config. What do you think?

Also, would you mind fixing the linting errors? If you don't have time let me know. If you could then allow me access to the pr I'd be happy to help.

[alexander-akait]

@ismay sorry, no time now for this, maybe late 😞

[ghost]

sorry, no time now for this, maybe late

No worries, linting errors are fixed 👍 Just ignored the no-param-reassign error btw, since I think the current style makes it clear what's going on. Also, we should be removing it later anyway, with the shareable config.

[coveralls]

Coverage increased (+0.02%) to 99.225% when pulling 173ee90 on evilebottnawi:patch-1 into 9842e3e on styled-components:master.

[ghost]

I'll try and rebase it this weekend btw, so we can get this merged.

[coveralls]

Coverage increased (+0.01%) to 99.225% when pulling 86fe76a on evilebottnawi:patch-1 into 68b4c4f on styled-components:master.