-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Custom file system support? #35
Comments
Hi @nathanhammond. Thank you for your interest in the library. Sure, that would be great. I assume the PR would add an interface which defines all permitted file system operations and a 'local fs' implementation of that interface, which would be the default? |
If that's the implementation that you feel like makes the most sense, then, yes! I'll get you a draft PR for your consideration up shortly. |
Sorry, I was just wondering if you were thinking of something along those lines 😄. |
@nathanhammond are you working on it? Because I'm also interested in it hehe. |
I was mulling over porting an internal tool's config system to use more standard directories over an upcoming lull and might be able to pick this up if wanted by @adrg .
I've done relatively little research thus far, but noticed that the open pull Seeking input on some possibilities (very rough sketch):
Uncertain how much performance penalty the added indirection would cause, and so feels like unit/performance tests are warranted. Just wanted to get your thoughts before spending addl time evaluating it next month. |
Hello @adrg! Would you be interested in a pull request that enabled custom file system support? (e.g.
afero
)The text was updated successfully, but these errors were encountered: