-
-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add rootDirectory
option
#35
Comments
Earlier @aminya stated that they want to be able to manually clear temporary objects created by |
I respect keeping the surface area small, but I'm pretty surprised that you are skeptical re its value. Many people who need or have cases for this might not want to comment publicly on why (private projects, NDAs, non-confrontational temperaments, etc). Added to which software is going to be used in contexts the creator doesn't anticipate. An example that comes to mind immediately (not my case) is that a different filesystem might be needed for these temporary files (e.g. I imagine most just use something else that gives them more filesystem control (even if by a less trusted name in software ™️). So you're not hearing about this from them because they just go someplace else 😸 . But obv your lib & call o/c. Just my 2 cents that is parameterizing the directory seems on the surface to provide value at minimal maintenance cost (plus I think there was already a 10-line PR someone did for this). |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
#46 would solve the use-case of grouping. |
I'm still willing to add an option called |
As requested in #33.
I guess it could be useful for when you want to semantically group all the temp directory usage in one directory for easier manual cleanup. This is not really useful for the
.task
methods though as they clean up themselves.I'm not entirely sold on this though, as every option adds complexity for the user and most users will not need this. I'm also concerned users will think they need this when most often, they don't.
The text was updated successfully, but these errors were encountered: