search

matype / stylefmt

stylefmt is a tool that automatically formats stylesheets.
Other
2.1k stars 89 forks source link

stylefmt -R not going through all files #127

Closed JaKXz closed 8 years ago

JaKXz commented 8 years ago

My understanding is that running this command:

$(npm bin)/stylefmt -R client/src/**/*.css

should affect all the css files in my client/src folder, but, only one file gets changed. There are definitely other files with fixable lint issues, but none of them are affected. Am I missing something?

matype commented 8 years ago

Tkank you for your report. I'll confirm it.

chrisblossom commented 8 years ago

I think I am having the same issue. https://github.com/morishitter/stylefmt/issues/89

matype commented 8 years ago

@JaKXz @chrisblossom Just released stylefmt v3.2.1 includes the commit fixed this bug. Please update and try it again.

chrisblossom commented 8 years ago

@morishitter thank you for fixing this. I got it working with the following command:

$(npm bin)/stylefmt -R src/

If i tried specifying the extension or the typical */ syntax it did not work. Just a note for anyone else reading this and @JaKXz

matype commented 8 years ago

Oh, I understood. Now, stylefmt dose not support to point directories by wildcard. I will consider to support it.

chrisblossom commented 8 years ago

@morishitter It would be consistent with most other libraries, including Stylelint and mocha to support the glob syntax. The current syntax works great for me, as long as it only changes *css files I am okay with it.

Thank you again. I appreciate the time and help.