search

flyntwp / flynt-cli

CLI tool for the WordPress Flynt framework
MIT License
14 stars 3 forks source link

TMP folder is not deleted #1

Closed dgrdl closed 7 years ago

dgrdl commented 7 years ago

tmp/flynt-cli folder persists as an empty directory after all tasks are done

domtra commented 7 years ago

so we should delete it if it is empty? it is gitignored, right?

dgrdl commented 7 years ago

Yes, or is there any benefit in keeping it around? There could be a cleanup task that runs after all flynt tasks are done / before exit and takes care of such things.

dgrdl commented 7 years ago

I just noticed that there is already a rm -rf running, although only on the ./tmp/flynt-cli/uploads folder. If we want to remove the tmp folder entirely, this needs to be adjusted.

If we want to keep it around, we need to at least add it to the adjustGitignore task since Bedrock does not include this in its own gitignore.