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

Add missing initial compile step. #280 #281

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add missing initial compile step. #280 #281

wants to merge 1 commit into from

Conversation

bricksphd
Copy link

When following the "How to run locally" instructions, the extension ran, but when called "Quick Input Samples," I received errors about missing files. I was able to fix this by run the compile script and then the Run Extension target.,

When following the "How to run locally" instructions, the extension ran, but when called "Quick Input Samples," I received errors about missing files. I was able to fix this by run the compile script and then the `Run Extension` target.
@Tyriar
Copy link
Member

Tyriar commented Apr 17, 2020

This shouldn't be needed since Run Extension triggers npm watch. Maybe node is not on the path causing the task to fail?

@chrmarti
Copy link
Contributor

This works for me. What files were missing and which npm version are you using?

@chrmarti chrmarti added the info-needed Issue requires more information from poster label Apr 21, 2020
@bricksphd
Copy link
Author

Exact errors in the extension VS Code window:
After Control-Shift-P -> Quick Input Samples

"Command 'Quick Input Samples' resulted in an error (command 'samples.quickInput' not found)"
"Activating extension 'vscode-samples.quickinput-sample' failed: Cannot find module '[path]\vscode-extension-samples\quickinput-sample\out\extension.js'
Require stack:

  • c:\Program Files\Microsoft VS Code\resources\app\out\vs\loader.js
  • c:\Program Files\Microsoft VS Code\resources\app\out\bootstrap-amd.js
  • c:\Program Files\Microsoft VS Code\resources\app\out\bootstrap-fork.js."

Environment:
OS: Windows 10 Pro
Terminal: wsl
node -v: 10.18.0
npm -v: 6.13.4

Base automatically changed from master to main February 22, 2021 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants