-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Evaluate if we should add Sorbet #62
Comments
@vinistock How did we decide to add Sorbet? (To be clear, I'm all for it, just curious what factors went into the decision.) |
Our default is to use Sorbet in all projects unless there's a good reason not to. It provides better safety with a super short feedback loop, better editor support among other things. Initially, I was pushing back on adding it because I had two concerns:
However, I believe the advantages are greater than these two. Because the Ruby LSP is used as a standalone executable, the addition of |
Awesome, thank you! |
Let's close this since we already decided to add it: #119. |
…t-eslint/parser-5.21.0 Bump @typescript-eslint/parser from 5.20.0 to 5.21.0
Evaluated the advantages and disadvantages to add Sorbet to the
ruby-lsp
, so that we can decide on whether we want to do it or not.If the decision is to add Sorbet, add it using our default tooling.
The text was updated successfully, but these errors were encountered: