CC-8419: make directory snapshot id optional - #7039
Conversation
|
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.46% -0.35%
==========================================
Files 468 457 -11
Lines 132296 127853 -4443
Branches 21474 20905 -569
==========================================
- Hits 89708 86256 -3452
+ Misses 29518 28970 -548
+ Partials 13070 12627 -443 ☔ 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.
If snapshot is empty, directory will just be prepared to be snapshotted upfront Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>
d029ef6 to
fe6a0bd
Compare
4 participants
If snapshot is empty, directory will just be prepared to be snapshotted upfront
Draft until internal mr merged
cc @spahl @rushilmehra