You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@sandersn That example doesn't type-check (and goto-def doesn't work for me): module.exports = { e: number, m: number }; fails because number is not a value, and /** @type {Foo} */ fails because Foo was never imported. Also, I thought a comment that doesn't precede anything was ignored?
Sorry, I typed up the repro based on code in a test buffer. I fixed the type errors.
I observed the problem with import('./mod1').Foo; I don't know if renaming Foo should work for `const mod1 = require('./mod1'); /** @type {mod1.Foo} */. Probably yes?
We have special-case code in the binder for typedef that is supposed to grab every typedef, even at the end of a file. I think they attach to the end of file node or something.
Expected behavior:
Renaming
Foo
in use.js should rename it. Find-all-refs onFoo
in use.js should list all referencesActual behavior:
Neither work. However, goto-def does work.
The text was updated successfully, but these errors were encountered: