-
Notifications
You must be signed in to change notification settings - Fork 57
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
archiver: rework implementation #425
Conversation
f08d2c7
to
7118f96
Compare
7118f96
to
149a431
Compare
@Dr-Emann Do you plan to review this? If not, I'll make a review myself as I'm willing to merge this soon... |
I don't know that I'll have a chance to review, unfortunately, sorry. |
No worries. I know it's a big change, but I think it's worth as it will make the archiver steps more clear and will allow you to implement something like #395 on top. In general, it allows to decouple the backup source from the archiving process... |
149a431
to
8c7318d
Compare
rebased. |
8c7318d
to
752523f
Compare
752523f
to
fbe7984
Compare
fbe7984
to
2e14f78
Compare
I did a review and some simplifications of the complicated types. Now IMO this is mergeable :-) |
No description provided.