Skip to content
This repository was archived by the owner on Feb 23, 2021. It is now read-only.

Add jsconfig. #1331

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add jsconfig. #1331

wants to merge 2 commits into from

Conversation

emiljoha
Copy link

@emiljoha emiljoha commented Feb 1, 2020

No description provided.

Copy link

@bolatovumar bolatovumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't node_modules folder be added to exclude property? See here: https://code.visualstudio.com/docs/languages/jsconfig#_best-practices

@emiljoha
Copy link
Author

emiljoha commented Feb 1, 2020

@bolatovumar Excluding node_modules folder sounds very reasonable and is supported by the vscode docs. Fixed in 4125e9a

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants