Skip to content

Commit a1ec37a

Browse files
committed
Note on SSH removal
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
1 parent f9f6f9a commit a1ec37a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

posts/2026-04-16-evolving-our-code-review-bot-with-slicer-sandboxes.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -276,12 +276,14 @@ The updated steps:
276276

277277
1. **Clone repo** - Short lived token minted using GitHub App's private key, no credentials in the VM
278278
2. **Launch and wait** - no polling, one SDK call, agent ready immediately
279-
3. **Copy in** - binary-safe tar stream via `CpToVM`, no SSH needed
280-
4. **Run with streaming output** - `Exec` with real-time stdout/stderr via channels, no SSH
279+
3. **Copy in** - binary-safe tar stream via `CpToVM`
280+
4. **Run with streaming output** - `Exec` with real-time stdout/stderr via channels
281281
5. **Watch for results** - event stream, instant notification
282-
6. **Read file directly** - `ReadFile` from the VM, no SSH
282+
6. **Read file directly** - `ReadFile` from the VM
283283
7. **Comment on PR** - post the review, then tear down the VM
284284

285+
In short, the workflow is more event-driven, and the SDK does the orchestration, instead of SSH.
286+
285287
## Wrapping up
286288

287289
We've been running this updated workflow on every PR across the company for months now. A fast, economic model reviews the diff before a human teammate gets to it - catching issues early so the human reviewer can focus on the harder judgement calls.

0 commit comments

Comments
 (0)