CC-8419: make directory snapshot id optional - #7039
Conversation
If snapshot is empty, directory will just be prepared to be snapshotted upfront Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>
|
Model not found: cloudflare-ai-gateway/anthropic/claude-opus-4-6. Did you mean: anthropic/claude-opus-4.5, anthropic/claude-opus-4.6, anthropic/claude-opus-4.7? |
|
@flakey5 Bonk workflow failed. Check the logs for details. View workflow run · To retry, trigger Bonk again. |
This comment was marked as low quality.
This comment was marked as low quality.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7039 +/- ##
==========================================
- Coverage 67.80% 67.37% -0.44%
==========================================
Files 468 456 -12
Lines 132296 126923 -5373
Branches 21474 20734 -740
==========================================
- Hits 89708 85511 -4197
+ Misses 29518 28930 -588
+ Partials 13070 12482 -588 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
|
||
| // Give the queued start RPC bounded time to run. | ||
| for (auto i = 0; i < 10; ++i) { | ||
| co_await kj::evalLater([]() {}); |
There was a problem hiding this comment.
If I remember correctly there's a kj::yield() that you could use, I think
jasnell
left a comment
jasnell
left a comment
There was a problem hiding this comment.
FYI, AI-reviewer is wonky right now so likely won't get a result from that.
4 participants
If snapshot is empty, directory will just be prepared to be snapshotted upfront
Draft until internal mr merged
cc @spahl @rushilmehra