Keegan Carruthers-Smith

Results 154 comments of Keegan Carruthers-Smith

I didn't get around to this yet, I did spike some time on it though. Is this still a problem for you?

Are you sure zoekt-index is using that version of ctags installed from that script? We use that in CI so our CI would be failing if that was the case....

I took a quick look at this and have a potential solution at https://github.com/sourcegraph/zoekt/pull/569. However, I don't fully understand the nitty gritty of the parse functions so needs more testing/review...

Hello @binarymason. Zoekt uses mmap, so depending on how you measure memory usage this might not be representative of the go heap but rather the file cache that the kernel...

> We are currently using this for calculating memory saturation: > > `container_memory_working_set_bytes / kube_pod_container_resource_limits ` If I am not mistaken this will include memory that can be evicted. To...

Here is a "simpler" solution which I hope fulfills a large number of use cases. I understand though it is probably better to do a proper solution which is what...

FYI I may be going about this completely the wrong way. I am experimenting with a workflow which takes a bunch of github PR inline comments and automatically guides the...

FYI this PR unblocks me using rules_ts on my NixOS development machine. Would be great to remove the `single_version_override` in our work repo :)