tenzinhl comments

Results 4 comments of


                                            tenzinhl

ModuleInfoCache.update_remote() should be elevate the contents of the HttpError rather than throwing them away

I was originally going to write a PR to make this suggested change, but since I'm a new user to OpenCRAVAT (and thus even greener of a dev) wasn't sure...

ModuleInfoCache.update_remote() should be elevate the contents of the HttpError rather than throwing them away

Hey! Sorry for the late update. PR is open now: https://github.com/KarchinLab/open-cravat/pull/123 Not many changes there, just has the exception bubble up. Places where the method is called will probably need...

eslint-import-resolver-node's resolution does not match node's with respect to symlinks

Awesome, thank you for this issue @anomiex , this was incredibly helpful. Our monorepo uses bazel with rules_js (https://github.com/aspect-build/rules_js) which uses pnpm for package management. Irrespective of bazel, I had...

Why is gen_connect_es deprecated?

Hey Antspy, my guess would be that this is because the new version 2 of `Connect-ES` no longer uses a separate `protoc-gen-connect-es` plugin when generating Connect code (https://buf.build/blog/connect-es-v2#:~:text=for%20these%20APIs.-,Simplified%20code%20generation,-Connect%2DES%202.0). That's to...