search

HaxeFlixel / flixel-docs

Documentation for HaxeFlixel
72 stars 135 forks source link

Haxeflixel clean uses the ls command #217

Closed Starmann closed 6 years ago

Starmann commented 6 years ago

Parsing ls is considered a shell scripting anti-pattern. It's recommended by most advanced shell scripters to use the find command instead.

Gama11 commented 6 years ago

What "Haxeflixel clean" command are you talking about?

Gama11 commented 6 years ago

I guess you mean https://github.com/HaxeFlixel/flixel-docs/blob/master/api/dox-gen/clean.hxml?

I'm not really sure why using using ls like would be an issue (appears to work fine), and it's mostly for "internal use" (for me to regenerate the API docs). Feel free to submit a PR if you want.