-
Notifications
You must be signed in to change notification settings - Fork 892
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
[red-knot] follow simple assignments #11181
Conversation
let parsed = db.parse(file_id); | ||
let ast = parsed.ast(); | ||
let node = node_key.resolve_unwrap(ast.as_any_node_ref()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see how resolving definitions might become expensive 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it would be awesome if we could directly reference AST nodes instead of having to search to resolve :/
|
You know have to label your PRs or they'll show up in the changelog 😆 |
Ugh, yes I know, I did remember to label one of them today! #11176 |
I think I would have remembered before merging :) |
d47e9ee
to
c21002d
Compare
64620b5
to
42eec9c
Compare
Summary
Follow simple assignments in type resolution.
Test Plan
cargo test