-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
docs: add sorbet documentation #8386
Conversation
91066f8
to
2c1e5d0
Compare
Going to add another section which deals with how to resolve type errors reported by Sorbet ASAP. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great start!
I'd like to see a bit more of an introduction here. Having some links to the Sorbet documentation and its "how to get started" documentation would be great. We want to make sure that Homebrew developers who are familiar with the Homebrew codebase, but not with Sorbet, have a place to get some guidance on where to go from here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me so far! Can you also link this in the README under "Contributors"? Thanks!
docs/Typechecking.md
Outdated
natively. RBI files for all gems are auto-generated using | ||
[Tapioca](https://github.com/Shopify/tapioca#tapioca). We can also create a RBI | ||
file to help Sorbet understand dynamic definitions. | ||
For eg: Sorbet assumes that `Kernel` is not necessarily included in our modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For eg: Sorbet assumes that `Kernel` is not necessarily included in our modules | |
For example: Sorbet assumes that `Kernel` is not necessarily included in our modules |
240fe22
to
eb137e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits otherwise: looks great to me, nice work!
(cherry picked from commit 022571489a0f94ed54a0908167b553592cfe3277)
eb137e5
to
cba751a
Compare
Let's merge for now and can always address additional comments in a follow-up PR. Nice work again @vidusheeamoli! |
brew style
with your changes locally?brew tests
with your changes locally?Documentation for Sorbet.