Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Sorbet ruby linter and fixer #2614

Merged
merged 2 commits into from Jul 13, 2019
Merged

Conversation

rintaun
Copy link
Contributor

@rintaun rintaun commented Jun 22, 2019

Adds support for the Sorbet type checker for Ruby that was just released publicly. I basically used the rubocop linter as a guide, along with a couple other LSP linters, so I'm not 100% sure that I did everything (or anything? 馃槀) in the best way -- especially the tests.

Thanks!

Copy link
Member

@w0rp w0rp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this, It all looks good to me! Sorry about the delay. I've been busy for a while.

@w0rp w0rp merged commit abb3895 into dense-analysis:master Jul 13, 2019
@w0rp
Copy link
Member

w0rp commented Jul 13, 2019

Cheers! 馃嵒

timlag1305 pushed a commit to timlag1305/ale that referenced this pull request Nov 5, 2019
@will
Copy link
Contributor

will commented Sep 22, 2020

@rintaun Thanks for this, I've been using it a lot. I was wondering though why you're using --disable-watchman ? I don't know much about watchman but maybe it could make the sorbet lsp pick up changes faster if it was on?

@rintaun
Copy link
Contributor Author

rintaun commented Sep 22, 2020

@will Honestly, I don't remember exactly, but as I recall it was necessary for it to work at all. That may have changed, as this was made quite a while ago, and I haven't been keeping up with the development of Sorbet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants