-
-
Notifications
You must be signed in to change notification settings - Fork 106
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: Do not store media in local project directory #4
Comments
Hey there, Thanks for checking out the project, and great points! I did consider this but for the initial release didn't have the time to implement something besides the I think for starters a configurable global media directory is a good option. This could be stored and read from a user-level or global-level configuration file similar to how Git's config works. The only real question in my mind is whether media generated from different repos should all go into the same folder, or whether there should be separate subfolders within the global media folder on a per project basis. Probably the latter would make more sense... Any interest in helping to implement something like this? Happy to give you the lay of the land of the Python codebase if it's something you're interested in poking at! And you would get the honor of being the first contributor! (Whatever that's worth :D) |
If the OP is not available to work on this, I'd like to give it a try and implement this. I agree with your suggestion for a global media dir followed by repo sub folders. Maybe the user can set an environment variable for the same ? |
Hey there! Sure honestly there's no queue so if you want to try implementing something feel free to do so, test it locally, and submit a pull request for me to check it out. Environment variable would be a cool addition too! |
Hey @initialcommit-io , would def like to work on this. Can you please add a contributions.md to let us know how to contribute/get this setup locally if possible? I think that would be of great help. Thank you!. |
Oh yes @PaulisMatrix good idea, will do that shortly! |
Hey @initialcommit-io
Let me know your thoughts |
A workaround I use is to define an alias that calls |
@abhijitnathwani I like that solution. It is less complex than having to worry about setting up config files at different scopes, like repo scope / user scope / system scope. I think it's a reasonable assumption for now that users who want to customize this location would be ok with a single global folder which would contain subfolders for each repo like you mentioned. That would also keep them nicely organized in one place, and I think is a good use case for an environment variable. Just 2 things:
Let me know if that sounds ok! |
@PaulisMatrix FYI I just added |
Thanks so much for making this happen! 🙌 |
First off, this is a very cool project. Thank you for it!
One thing I don't like is that git-sim stores it's media files in the local project directory, and it's easy to accidentally commit them to the git repo if it hasn't been added to the .gitignore file. Most of the time I think the images/videos are only needed temporarily, and could be deleted. There is the
--media-dir
parameter, but having to provide it every time you use git-sim is cumbersome.There's a few different ways you could tackle this problem. Some thoughts I had were:
Maybe you (or others) have other suggestions?
The text was updated successfully, but these errors were encountered: