Skip to content
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

No IntelliSense for Express in MERN starter kit #23317

Closed
chrisdias opened this issue Mar 27, 2017 · 2 comments
Closed

No IntelliSense for Express in MERN starter kit #23317

chrisdias opened this issue Mar 27, 2017 · 2 comments
Assignees
Milestone

Comments

@chrisdias
Copy link
Member

  • VSCode Version: Insiders 1.11.0
  • OS Version: Mac OS

Steps to Reproduce:

  1. scaffold out new MERN app:
    npm install -g mern-cli
    mern init myApp
    cd myApp
    npm install
    code-insiders .
  1. Open server/server.js
  2. After line 15 type in app.

image

Result: textual completions (you don't get completions for Express)

  1. Now, change import to lowercase express and change instantiation to remove new.

image

Result: IntelliSense works.

@egamma
Copy link
Member

egamma commented Mar 30, 2017

@chrisdias This is a bug in the starter kit. According the express API documentation http://expressjs.com/de/api.html#express you create an express app by calling the top level express() function. This is a top-level function and not an object constructor and it therefore should be called as a function.

I suggest to make a PR against the MERN starter kit.

@chrisdias
Copy link
Member Author

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants