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
fix: surface errors when compressing files #13491
fix: surface errors when compressing files #13491
Conversation
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #13491 +/- ##
=======================================
Coverage 49.20% 49.20%
=======================================
Files 248 248
Lines 42908 42908
=======================================
Hits 21112 21112
Misses 19682 19682
Partials 2114 2114
☔ View full report in Codecov by Sentry. |
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.
LGTM
/cherry-pick release-2.6 |
/cherry-pick release-2.7 |
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…#13494) Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
CompressFiles
returnsfilesWritten == 0
for all cases whereerr != nil
, but whenerr != nil
, the fact that no files were written isn't the most interesting bit of information. i.e. we're hiding the actual error in many cases.This change will pass up the error message if there is one. If there isn't an error, we throw a new error if no files were written.