-
Notifications
You must be signed in to change notification settings - Fork 155
Open
Description
I've been experimenting with the workflow and wanted to document a few issues that have come up already.
Cases not handled by spoom
- abstract and interface - spoom does not convert these signatures. There are not usually a lot of these. They can be converted manually. However, in the essentials migration we sometimes found these annotations were not accurate. This may be fixed at this point in sorbet, but these will be worth watching.
- it has been suggested that we delay these until after we have a better solution for abstract methods, however, that appears to be a significant blocked.
- RBS does not have support for singleton class type: https://github.com/Shopify/team-ruby-dx/issues/1460
Other considerations
- RuboCop and styling - There may be style failures that are unrelated to the code being changed. When checking styles with
--include-branch-commitsthis may be checking files that have not been checked recently and style rules have changed.- Should we style with the RBS changes or cleanup styles separately, possibly before-hand?
Metadata
Metadata
Assignees
Labels
No labels