open-wc / custom-elements-manifest

Custom Elements Manifest is a file format that describes custom elements in your project.

Home Page:https://custom-elements-manifest.open-wc.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

The analyzer doesn't ignore nested node_modules by default

andrico1234 opened this issue · comments

Checklist

I'm using the CLI to generate a CEM for a monorepo.

For large monorepos, the analyzer hangs for a long time, since it's analyzing files inside of nested node_modules.

You can see in the repro that the output contains modules that live within node_modules directory

This suggests that by default it doesn't ignore nested node_modules. After looking into the source it seems that the default ignore pattern for node_modules is "!node_modules/**.*". My guess is that this ignores the top-level node modules. By adding the following pattern "**/node_modules/**" the analyzer avoids going through the nested modules

Completing the items above will greatly improve triaging time of your issue.

Expected behavior
Ignore node_modules at all levels by default

I'll be happy to create a PR if this is something that you feel should live in the repo

Can you create a pr?