search

mrmlnc / vscode-stylefmt

:electric_plug: VS Code plugin for stylefmt— Format your CSS using stylefmt.
https://goo.gl/pdJJBD
MIT License
45 stars 3 forks source link

Autofixing on errors #96

Closed svipas closed 4 years ago

svipas commented 7 years ago

Hello,

I know that stylelint still not released yet with --fix feature, but after release when also stylefmt will be updated to use latest version of stylelint, what do you think about fix feature on format?

If you can don't close this issue, let it be, when stylelint and stylefmt will be updated with --fix feature we can discuss here to include it or no.

mrmlnc commented 7 years ago

IMHO, this does not apply to this plugin. This should be implemented in the stylelint plugin.

svipas commented 7 years ago

@mrmlnc You're saying true, but stylelint plugin owner doesn't give a **** about that. By the way prettier is JS formatter, but it adds needed semicolons and so on, it works like --fix, but with extra addition, I was thinking the same for this CSS formatter, but anyway, you're saying true, this function not belongs for stylefmt. You can close issue if you want.

mrmlnc commented 7 years ago

You must to understand that they are two different entities and they are not associated. This functionality applies to Stylelint. Ideally, Stylelint must absorb Stylefmt because it uses the rules first. We can fork or send PR's to vscode-stylelint. I'm not going to close the issue to remember about him.

mrmlnc commented 4 years ago

Fixed by #162.