search

Urthen / case-sensitive-paths-webpack-plugin

Enforces case sensitive paths in Webpack requires.
MIT License
428 stars 45 forks source link

Add Include/Exclude globs to options, because it is so slow #42

Open AuthorProxy opened 4 years ago

cdeutsch commented 4 years ago

Wow, I just went from 3 minutes to 2 minutes by removing CaseSensitivePathsPlugin 😳

It is a bit slow.

Urthen commented 4 years ago

If you're having slowness problems, can you run with the {debug: true} option and let me know how many filesystem operations it is performing? That'll help me figure out what kind of scale project you're working on.

As for adding include/exclude globs, I probably won't add that as it would largely defeat the purpose of the plugin (which is to detect case sensitivity errors wherever they occur). However, if you are working on a project with TONS of node-modules dependencies, I think it might be safe to add an option to skip those, as it is much less likely to run into accidents there.

cdeutsch commented 4 years ago

@Urthen Total filesystem reads: 13,189

calling commented 4 years ago

We're experiencing a similar issue. Total filesystem reads: 15057