forked from sorbet/sorbet
-
Notifications
You must be signed in to change notification settings - Fork 5
[Follow up] Clean-up tasks to do once direct desugaring is complete #671
Copy link
Copy link
Open
Milestone
Description
Once we've fully finished implementing direct desugarings in Translator.cc (#444), we can clean up a whole bunch of stuff. Use this issue to track such tasks.
- Remove all usages of Whitequark nodes
- Delete
PrismDesugar.cc, and renameTranslator.ccto be the newPrismDesugar.cc- Opened as , but merged in Hide the Translator class in the
ccfile sorbet/sorbet#10000
- Opened as , but merged in Hide the Translator class in the
- Stop treating unsupported nodes as if they are
self - Change
desugarScopeBodyToRHSStoreto not returnstd::optional - Stop skipping tests such as sorbet@9feb9fb exclusions in
test/BUILD - Update this comment to not reference
parser::Arrayandparser::Send - Narrow the return type of this function and others which had their types widened
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Type
Fields
Give feedbackNo fields configured for issues without a type.