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

VS Code Error due parsing too many files #9972

Closed
jardakotesovec opened this issue May 21, 2024 · 3 comments
Closed

VS Code Error due parsing too many files #9972

jardakotesovec opened this issue May 21, 2024 · 3 comments

Comments

@jardakotesovec
Copy link
Contributor

jardakotesovec commented May 21, 2024

Describe the bug
Recently I was getting following error in vs code:


To enable project-wide JavaScript/TypeScript language features, exclude large folders with source files that you do not work on.

As result some JS features did not work on some files. Per my testing the root cause was that the jsconfig was just excluding top level node_modules. Solution that is working well is to point jsconfig to src folders.

Solution
Refine jsconfig.json to make sure vs code is dealing just with the source files

What application are you using?
OJS, OMP or OPS main

jardakotesovec added a commit to jardakotesovec/ojs that referenced this issue May 21, 2024
@jardakotesovec
Copy link
Contributor Author

OJS PR: pkp/ojs#4279

Will add PRs for OMP and OPS after review.

@ewhanson Could you have a look?

jardakotesovec added a commit to jardakotesovec/ojs that referenced this issue May 21, 2024
@ewhanson
Copy link
Collaborator

Just one question in the PR, but otherwise looks good! 👍

jardakotesovec added a commit to jardakotesovec/ojs that referenced this issue May 22, 2024
jardakotesovec added a commit to jardakotesovec/omp that referenced this issue May 22, 2024
jardakotesovec added a commit to pkp/omp that referenced this issue May 22, 2024
jardakotesovec added a commit to jardakotesovec/ops that referenced this issue May 22, 2024
jardakotesovec added a commit to pkp/ops that referenced this issue May 22, 2024
@jardakotesovec
Copy link
Contributor Author

Thank you, added couple more paths and added jsconfig to
omp - pkp/omp#1569
and ops - pkp/ops#683

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants