-
Notifications
You must be signed in to change notification settings - Fork 2
Description
It appears that this extension is now broken, and unmaintained (last change occured 2 years ago, at the time of writing this).
- Not working, with example:
Consider this basic CommonJS (require()) syntax for importing the Express web framework:
var express = require('express');
var app = express();
app.set('port', process.env.PORT || 3000);With the cursor on the last line's set symbol, calling the Go to definition (F12) VSCode function, gives the error No definition found for 'set'.
Instead, it should resolve the node_modules directory hierarchy (located in the same directory as the file with above snippet).
Specifically it should resolve the /node_modules/express/lib/application.js file, on the line:
app.set = function set(setting, val) {Mind you I have disabled the TSServer with:
"typescript.disableAutomaticTypeAcquisition": true
in order to disable the default behaviour which sends theg user to the index.d.ts file upon Go to definition.
- Documentation on README is misleading when there is no 'Get Started' section:
Still confused? Click "Get Started" above.