autorestic icon indicating copy to clipboard operation
autorestic copied to clipboard

Refactoring

Open rdelaage opened this issue 2 years ago • 4 comments

Hi, I was trying to implement #256 and was thinking that it could be a good idea to make some refactoring to the code base to maintain it more easily. Especially it could be a good idea to create some modules like config to handle all configuration thing or a logger module, add some generic functions were necessary, ...

Do you think it is a good idea ? I would be happy to work on this :)

rdelaage avatar Nov 13 '22 15:11 rdelaage

If we want to change things, I don't think the current idea of location with the "normal" and the volume type is the best. I think we can have a only type of location and volumes can be part of the from option with a special kind of prefix (e.g. volume://<name>) and optionally a path in this volume (e.g. volume://<name>/a/very/long/path)

rdelaage avatar Nov 14 '22 16:11 rdelaage

Hey! Thanks for all the input! I was also in the means of refactoring, but I won't find time until january unfortunately. I see you have a lot of ideas, what about a short call the coming days?

cupcakearmy avatar Nov 15 '22 12:11 cupcakearmy

Hey! Why not, I am registered on your Discord channel if you want. I also put my thought on this document

rdelaage avatar Nov 15 '22 14:11 rdelaage

Awesome! I've messaged you on discord :)

cupcakearmy avatar Nov 15 '22 15:11 cupcakearmy