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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Ruby sorbet option to enable watchman #3403

Merged
merged 1 commit into from
Dec 16, 2020

Conversation

will
Copy link
Contributor

@will will commented Oct 21, 2020

Follow up to #2614

The sorbet support already in ale works great, and I think it makes sense to default to --disable-watchman as @rintaun did in the original PR.

A little while ago I had hacked up my copy in my .vim dir to remove that though, so files changed outside of vim (such as auto generated rbi files) would be picked up by the lsp server.

I figured it'd be better to contribute that back up to ale though. I've done very little vim script though, so this was all just done by looking around the project a bit, so I'm not sure if things are wrong. It seems to work for me though, for what it's worth.

@will
Copy link
Contributor Author

will commented Oct 21, 2020

It's cool that it all works so fast, even though there are many moving pieces. Thanks for making ale quick :)
vimalelsp

@will
Copy link
Contributor Author

will commented Nov 18, 2020

posting to keep that sale bot away

@stale
Copy link

stale bot commented Dec 16, 2020

This pull request has been automatically marked as stale because it has not been updated recently. Make sure to write tests and document your changes. See :help ale-dev for information on writing tests.
If your pull request is good to merge, bother w0rp or another maintainer again, and get them to merge it.

@stale stale bot added the stale PRs/Issues no longer valid label Dec 16, 2020
Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Looks good.

@stale stale bot removed the stale PRs/Issues no longer valid label Dec 16, 2020
@hsanson hsanson merged commit f996ede into dense-analysis:master Dec 16, 2020
@will
Copy link
Contributor Author

will commented Dec 16, 2020

Thanks!

@will
Copy link
Contributor Author

will commented Dec 16, 2020

1337 closed pr 😎
image

@will will deleted the sorbetwatchman branch December 16, 2020 00:52
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

2 participants