-
Notifications
You must be signed in to change notification settings - Fork 216
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
Improve core/stdlib signature coverage #1
Comments
Sorbet already has type signatures for a lot of the stdlib, have you considered writing something to transform their files into ruby-signature-compatible files to automate this process? |
Yes, it should be one of the best ways to improve the coverage. I haven't tried yet. |
I know this ended up partially happening in #23, do we have any way of tracking progress for stdlib coverage? Essentially code coverage for type signatures. :) |
I have a guide for contributions at https://github.com/ruby/ruby-signature/blob/master/docs/CONTRIBUTING.md . Could you check it? Tell me if you have any questions or feedback. |
Yes, it should be a good option to start having an initial version. However, I believe we need to review and revise them ourselves anyway. |
Hey, is this issue still open? I would like to work on this issue. |
Hi. Is it correct that this issue is still active? Also, have you already implemented the idea of importing the sorbet type definition file? I'm hesitant to start a contribution because I personally can't read the status of this issue. Thank you for your time. |
I' ve read the code and the PR, and it seems to be still open, so I'll try to help as much as I can.
It seems to have been addressed in the following PR 👍 |
I thought it would be better to change the title of this issue to "core and standard library". Reference : #876 |
We provide the signatures of a very limited number of standard library classes. We will really appreciate if you write the signature of other classes.
(Note that we are currently working for standard library signatures.)
The text was updated successfully, but these errors were encountered: