Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Can @types/electron be removed as a devDependency? #9

Closed
walkingriver opened this issue Jun 23, 2017 · 5 comments
Closed

Can @types/electron be removed as a devDependency? #9

walkingriver opened this issue Jun 23, 2017 · 5 comments

Comments

@walkingriver
Copy link

walkingriver commented Jun 23, 2017

I'm pulling ngx-electron into my project as a dependency, which has a dependency on @types/electron, which appears to have a dependency on electron. This causes my Angular app's node_modules to have a complete electron distribution in it, which it never uses. However, when I then use electron-builder, it dutifully pulls it in. According to this (https://www.npmjs.com/package/@types/electron), there is no reason to include @types/electron anymore. Should it be removed entirely?

@walkingriver walkingriver changed the title Should @types/electron be removed as a devDependency? Can @types/electron be removed as a devDependency? Jun 23, 2017
@walkingriver
Copy link
Author

For grins, I did try what I'm suggesting. I forked the repo and removed @types/electron from package.json entirely. Now it won't build, because the namespace Electron doesn't exist. I imagine there must be a way to do this without bringing in the entire electron package as an angular dependency. I just can't think of what that is.

@ThorstenHans
Copy link
Owner

@walkingriver electron provides it's own types since two weeks I think. They've deleted @types/electron as mentioned on npmjs.

I'm currently refactoring ngx-elecrton to work again correctly.

@walkingriver
Copy link
Author

Let me know if there's anything I can do to assist.

@ThorstenHans
Copy link
Owner

ThorstenHans commented Jun 23, 2017

thanks for the offer. I've just published 1.0.3 on npmjs perhaps can you try it in your scenario / build. I've tested and verified functionality with @angular/cli based samples.

@walkingriver

related to #10

@ThorstenHans
Copy link
Owner

@walkingriver thanks for the input I'm closing this due to the currently working 'workaround' 😄 and the upcoming release mentioned in #11

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

No branches or pull requests

2 participants