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 sorbet for type checking #1873

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Add sorbet for type checking #1873

wants to merge 10 commits into from

Conversation

Throne3d
Copy link
Contributor

@Throne3d Throne3d commented Nov 21, 2023

No idea if we'll ever merge this, just wanted to test out how it'd look + what it'd be like!

Update via:

  • bundle exec tapioca init
  • If we get new entries in sorbet/rbi/todo.rb, try adding entries to sorbet/tapioca/require.rb for those items, or if necessary create an entry in shims/gems.
  • bundle exec tapioca generate (previously gems)
  • bundle exec tapioca dsl
  • bundle exec srb tc - check type checking

Shopify/tapioca#1266

@Throne3d
Copy link
Contributor Author

There's a truly insane amount of vendored code here - TODO: gitignore sorbet/rbi (against the upstream recommendation) and just tell people to generate these files locally?

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