-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
[feature request] support default value for destination in options bag #55
Comments
See https://systemd.io/TEMPORARY_DIRECTORIES/ for guidance on avoiding pitfalls and https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap08.html#tag_08_03 for expected location on Linux systems as defined by an environment variable. |
Currently, it is not very well documented, but the options bag passed to the class constructor at initialization requires that a String be supplied for the location of the destination directory for the files/folders imported from upstream repos.
Firstly, rather than simply expecting a String, the type should be narrowed to perhaps only accept a valid filesystem path.
In the event that this field is omitted, such as in the case where the user may not necessarily be interested in the file(s) or may not have a particular preference where files are stored, a smart default for the canonical temporary directory should be used.
For Node.js environments with
node:os
core module available, the value returned by theos.tmpdir()
method may be useful to consider. In the case of browser environments, theWindow.sessionStorage()
Web API may be of interest.The text was updated successfully, but these errors were encountered: