-
Notifications
You must be signed in to change notification settings - Fork 34
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
Backup: don't ignore uploadHost error #3731
Conversation
I will try to add tests later today. |
This should be included in the 3.2.6 release. |
Please add the test and change the PR from draft to regular one. |
9e9b11a
to
fa7dccf
Compare
@karol-kokoszka I added a test and it turns out that it detected another, not connected issue #3733. I will tackle the other issue later into the week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one nit (it's about test code).
Other than that 👍
They were leftovers from times when restore and backup lived in the same pkg.
d3c1ea5
to
3e9a9de
Compare
@karol-kokoszka I decided to add a fix for failing tests in this PR. I also adressed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Fixes #3729
Fixes #3733