-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
Types issue with Glob and Minimatch #264
Comments
You shouldn't need those |
Confirmed, the
|
Thanks this does work, although it is still bit annoying since @types/glob is listed as an (optional) dependency for @electron/asar, so running npm install will put it right back. |
Sounds like @electron/asar needs to not have that optional dep. DT types should usually only ever be dev deps. Tell 'em I sent ya. |
I'd help more, but there's literally nothing I can do. I don't maintain electron, definitelytyped, or typescript. Fwiw I did request that they remove the @types/glob type, or make it a stub, because glob has its own types now. |
Thanks for your help! |
@isaacs Do you want to remove the @types/glob type? If I remove the glob folder , its regenerating the folder again. Do I need to make any changes to the package.json or tsconfig file? becuase I want to push the code our QA server without any issue. |
I have same issue with [email protected] that has dependency on @types/[email protected] but can't figure out how to solve it. Sorry not an expert at Angular and npm :) Can you please help me? |
Please follow the steps I detailed above. |
When I run rimraf dist I see:
Error is also noted here, and the solution is to not run rimraf:
https://stackoverflow.com/questions/75890950/node-modules-minimatch-dist-cjs-index-has-no-exported-member-ioptions
Minimatch, glob, and @types/minimatch and @types/glob are all at their latest version.
The text was updated successfully, but these errors were encountered: