search

asteris-llc / vaultfs

Vault filesystem (and Docker volume plugin)
Apache License 2.0
129 stars 13 forks source link

Docker driver, lint. #3

Closed ojongerius closed 8 years ago

BrianHicks commented 8 years ago

Why is this code doing what it's doing?

ojongerius commented 8 years ago

Sorry I could have given some context on this one.

That's a good question! I wanted go test to stop complaining and satisfied it by adding the volumName struct, Get and List functions. It's not tested, and as I'm not using the Docker driver (no support for custom volume drivers in ECS as of yet), who knows if it works as intended :blush:

Feel free to do with this one what you want, I can close the PR if you have no intentions of merging it. I'm keeping it around in my fork so I can run Travis and test other changes.

BrianHicks commented 8 years ago

Great fix, thanks :)