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

Option to disable dependency tracking #2905

Closed
novemberborn opened this issue Dec 5, 2021 · 2 comments
Closed

Option to disable dependency tracking #2905

novemberborn opened this issue Dec 5, 2021 · 2 comments

Comments

@novemberborn
Copy link
Member

Dependency tracking may be fraught in CommonJS and currently doesn't work anyway in ESM. See discussion in #2887 and #2388.

When dependency tracking breaks, watch mode becomes less useful. Let's add a trackDependencies configuration option, defaulting to true, which when set to false disables dependency tracking. We can add a guard here:

dependencyTracking.install(require.extensions, testPath);

Presumably the solution for #2388 will involve parsing syntax files which may be slow, so having an opt out will be useful regardless.

@Sarveshk76
Copy link

image

Is it right?

@novemberborn novemberborn self-assigned this Oct 23, 2022
@novemberborn novemberborn linked a pull request Oct 23, 2022 that will close this issue
5 tasks
@novemberborn
Copy link
Member Author

This would be irrelevant if #3123 is completed.

@novemberborn novemberborn closed this as not planned Won't fix, can't repro, duplicate, stale Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants