Keegan Carruthers-Smith

icon company @sourcegraph icon location Cape Town, South Africa icon location South African working for @sourcegraph. Did stints in Bay Area and London for @facebook.

Results 154 comments of Keegan Carruthers-Smith

Refactor zoekt-webserver flags parsing

> > There is already the `web` package which exists for this purpose, I suspect a bit too much has grown into the zoekt-`webserver` command? > > That's my impression...

Ink (CLI framework) not working properly

Filed this related issue. Ink breaks if you use a stdin created with `tty.ReadStream(openSync('/dev/tty', 'r'))` since that ReadStream doesn't have the `ref`/`unref` methods but `process.stdin` does. https://github.com/oven-sh/bun/issues/22372

fix: implement sql/driver.Pinger interface

Out of interest at Sourcegraph we work around this problem by wrapping the driver again and skipping over sqlhooks. https://github.com/sourcegraph/sourcegraph/blob/cab2c723c5600345a1714ee236ef726af40870b1/internal/database/dbconn/open.go#L89-L117

Add CudaText LSP Client

you are welcome to submit a PR and I will review and merge it.