-
-
Notifications
You must be signed in to change notification settings - Fork 358
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
fix(semantic): do not bind type/value references to value/type symbols #3863
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
CodSpeed Performance ReportMerging #3863 will degrade performances by 11.87%Comparing Summary
Benchmarks breakdown
|
Thanks for putting this together @DonIsaac. A couple of responses:
type Foo = number;
const Foo = 123;
function f(foo: Foo) { // `Foo` here refers to `type Foo`
return Foo; // `Foo` here refers to `const Foo`
} I think before going further, we should figure out what our goal is. |
For anyone reading this PR thread, the conversation is taking place on #3799. |
…ix/value-type-references
…ix/value-type-references
@Boshen and I are studying how TyepScript resolves references and what |
…thin interface heritage nodes
We are studying how tsc does this. Closing for house keeping. cc @Dunqing You may want to copy over some of the tests from this PR. |
Fixes #3799
Only symbols with the desired "meaning" (value/type) get resolved when binding references to symbol ids.
I also updated some code in
oxc_traverse
, but to be honest I'm not sure what's going on there and I'd love some input on what to do there. It seems to be used byoxc_semantic
, but not completely?References
createNameResolver
Todo