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

Read local RBI files when parsing #169

Closed
wants to merge 1 commit into from

Conversation

dorner
Copy link
Contributor

@dorner dorner commented Jul 19, 2023

This update adds the local sorbet directory to sord's list. Most gems don't have actual rbi directories in them - instead a lot of them depend on sorbet-typed. Using tapioca will dump a bunch of RBI files into a local sorbet directory, which sord can then read and use as types inside type declarations.

This does take a bit of time (the default is to actually generate types for every single gem that's included, which is likely overkill since very few of them would be directly referenced in the project code). Not sure if we want to print out some kind of message when we parse this directory indicating that the user might want to delete some of those auto-generated files to speed up the process.

@dorner
Copy link
Contributor Author

dorner commented Jul 19, 2023

Closing in favor of #170.

@dorner dorner closed this Jul 19, 2023
@dorner dorner deleted the read-local-rbis branch July 19, 2023 18:57
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.

1 participant