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
♻️ Better user feedback on folder-like artifacts #1589
Conversation
# we don't yet have logic to bring back the deleted metadata record | ||
# in case storage deletion fails - this is important for ACID down the road | ||
if delete_in_storage is True: |
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.
FYI: @sunnyosun - this is an anti-pattern!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1589 +/- ##
==========================================
- Coverage 90.32% 90.29% -0.03%
==========================================
Files 51 51
Lines 5341 5348 +7
==========================================
+ Hits 4824 4829 +5
- Misses 517 519 +2 ☔ View full report in Codecov by Sentry. |
No description provided.