-
-
Notifications
You must be signed in to change notification settings - Fork 166
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
Emit invalid error(want 2471, but 2322) #745
Comments
Not sure if this can be a GFI. In my experience 2322 vs 2741 was quite complex |
Im pretty sure this also depends on #741 |
It'd be nice to combine the two together, |
I tried this, but this requires implementing missing property logic for class assignments. |
Closed by #769 |
Do you know this test file? (
tests/conformance/types/typeRelationships/assignmentCompatibility/unionTypesAssignability
)Except for allocations related to the Union,
There is a difference in properties, so it should be 2471.
But stc emit 2322
because the logic returns 2322.
https://github.com/dudykr/stc/blob/main/crates/stc_ts_file_analyzer/src/analyzer/assign/class.rs#L299
I think that we should touch only
assign/mod.rs
andassign/class.rs
I think it's an easy issue except for coordinating the order.
The text was updated successfully, but these errors were encountered: