It makes sense to include logic to remove duplicate endpoints. I'll add that.
I've fixed the latest merge conflicts and will start working on the rest of suggestions. Development might still be…
@witten I had this concern too since I started working on this. I was also wondering if someone wants to receive notification for a failed event in any of 4 actions, he will have to set the…
Noted. I would personally prefer the skip-monitoring flag if I am configuring monitoring. I've removed the short flag but let me know if you rather want to go through your alternate suggestion.
@witten Hey, with start of GSoC and completing the other PR, this one slipped my radar.
So Sorry for the delay! I do intend to complete this and will get right back as soon as possible. I…
Thank you for bearing with me and your continuous help! :)
Last time I checked I thought I was getting other commits in my files changed. It looks fine right now though.
@witten I mostly merge with Github and seem to have messed up conflict resolving with merge as it's assigning other commits to this pull. Can you help?
@witten Sorry for the delay. Exams still going on 😅.
I've updated/fixed existing tests and added a test to prune for your review. Can you please take a quick look and I'll pull new commits…
@witten I did not encounter any merge issue.
Can you give me pointers on testing? I suppose we can add a date-based matching test for each subcommand under /tests/unit/borg. Is there anything…
Note that there's unfortunately been some work in master that will pose minor conflicts with this branch. Hopefully it won't be a big deal to merge those in.
No worries, I'll resolve…
I couldn't find a way to simplify it further which can also be read easily. This definitely requires one or two re-reads for first time. I'll check back on this again after tests.
I think it's ready for a review. Only tests should be missing.
I decided to go with calling ping_monitor in a loop.
@witten How do you want to go about adding flag to turn off monitor hook in interactive mode? Does borgmatic already have a method to detect cron or interactive mode?
If not, I think it'd…
Is length of help text a problem?
'--newest', metavar='TIMESPAN', help='Mount archives within a time range that ends at timestamp of the newest archive and starts a specified time range…
@witten I unexpectedly missed that multiple actions. Will fix it.
Is there any specific reason make_flags_from_arguments() isn't used on every action? I'm working on the PR and have left the borg files untouched which are used this function and passed all…