ashgti wrote:

> @ashgti Hey John, do you mind splitting this patch?

Sure, I'll make this fixing the package.json.

> As for the package.json, do we have any tests that try to npm build? Should 
> we add them?

I don't think they're run as part of the CI today. We could see if there is 
some way to integrate the npm build / tests into the lldb-dap CI somehow. 

https://github.com/llvm/llvm-project/pull/138918
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to