-
Notifications
You must be signed in to change notification settings - Fork 44
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
Restructure venus temp #350
Restructure venus temp #350
Conversation
…f .venus_temp-#PORTNUMBER# verses straight .venus_temp
…setPort, and some other one I forgot about, iono look at the code
…upport this change
What if we used the system temp directory and created a new folder for each run of venus, for example:
We could also add a Seth |
I think @sethmcl suggestion makes a lot of sense! |
Thanks @sethmcl! It's funny, I was initially thinking the same when it came to grouping them together. Well, the naming wasn't the same, but more the directory hierarchy. The one thing I didn't group by was the GUID, which I like. A question about that, when you say GUID, can you clarify that? Specifically are you referring to generating a GUID, getting the unique process id, or something else? Updating the logic to do this shouldn't be too difficult. There is already a good amount of changes in this one pull request. Therefore, I'd lean towards doing the |
I agree with saving For the guid, I was thinking we could just use a library like https://www.npmjs.com/package/node-uuid |
…down and strengthened some things
That sounds good regarding setting the guid. I got the structure in place for this so it'll be really easy to get that going. I'll add the guid, update the tests, and docs like you suggested within the next day or so. I hope it's not an issue, but when I saw how easy it was to add the |
…e the temp directory on the file path will be unique - the same thing is also achieved with the venus server
… its functionality
The squashed version of this can be seen here: #355 |
Here's a brief overview of what's happening: