Update TSRX to v0.0.85 - #7173
Conversation
|
We require contributors to sign our Contributor License Agreement, and we don't have @zhsama on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
We require contributors to sign our Contributor License Agreement, and we don't have @zhsama on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
MrSubidubi
left a comment
There was a problem hiding this comment.
Thanks for this!
We usually do not check for the server with a script like this - you can either just let Zed error in that case (we show a generic error there) or check with fs::metadata, then throw an appropriate error.
Would kindly ask you to make that update, all else looks good. Thanks!
|
Thanks! I removed the shell/cmd wrapper and switched the extension back to launching the resolved language-server executable directly in Ripple-TS/ripple#1416. Once that upstream change is merged, I’ll update this PR’s submodule pointer to the merged commit. |
|
we have a process now set up to publish the extension via a pr whenever one released. here's the one for version 0.0.86 #7200 which has all fixes in this pr + the merged pr that calls the language server directly: Ripple-TS/ripple#1416 |
|
Thanks! Closing in favor of the newer bump |
Summary
Why
The registry still points at TSRX 0.0.84. Ripple main already contains 0.0.85, which updates the language tooling to the current TSRX packages and includes Octane compiler auto-detection for published packages.
Validation