-
Couldn't load subscription status.
- Fork 100
fix: Fix broken ingredient recursion #1309
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
Conversation
…sh_updates # Conflicts: # sdk/src/assertions/labels.rs # sdk/src/store.rs
…sh_updates # Conflicts: # sdk/src/assertions/labels.rs # sdk/src/store.rs
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1309 +/- ##
==========================================
- Coverage 78.30% 78.28% -0.03%
==========================================
Files 153 153
Lines 39106 39157 +51
==========================================
+ Hits 30622 30653 +31
- Misses 8484 8504 +20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
looks good, there's a few places where we should be logging uris instead of labels.
Changes in this pull request
Updates to tweaks to hash binding logging
Fix accidentally removed unknown ingredient status
Better update manifest handling
Checklist
TO DOitems (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.