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

Unmet dependency electron #40

Closed
mrgrauel opened this issue Apr 27, 2018 · 6 comments
Closed

Unmet dependency electron #40

mrgrauel opened this issue Apr 27, 2018 · 6 comments
Labels

Comments

@mrgrauel
Copy link


Issue

  • After Update of the plugins hyper also show unmet peer dependency for hyperterm-summon electron

Steps to Reproduce

  1. open hyper
  2. update plugins and wait for the notifications

Expected Result

  • all dependencies are resolved

Actual Result

  • missing electron dependency
@dcalhoun
Copy link
Collaborator

  1. You replied "N/A" for "Relevant information from devtools". Was there a specific error message that you can screenshot or copy/paste?
  2. When you say "update plugins and wait for the notifications" is this from the "Plugins > Update" menu item or the auto-update that occurs every 5 hours?

@accessd
Copy link

accessd commented Apr 28, 2018

Same issue. Message appears automatically:

fullscreen_28_04_2018__22_03

~> npm install
npm WARN hyperterm-summon@2.0.8 requires a peer of electron@^1.6.2 but none is installed. You must install peer dependencies yourself

@dcalhoun
Copy link
Collaborator

Thank you for the additional information, @accessd. I'll prepare a fix now.

@dcalhoun dcalhoun added the bug label Apr 28, 2018
dcalhoun added a commit that referenced this issue Apr 28, 2018
Remove the unnecessary `electron` peer dependency. Fixes #40.
dcalhoun added a commit that referenced this issue Apr 28, 2018
Remove the unnecessary `electron` peer dependency. Fixes #40.
@dcalhoun
Copy link
Collaborator

@mrgrauel @accessd please let me know if this is resolved for you in the hyperterm-summon@2.0.9 release.

@accessd
Copy link

accessd commented Apr 28, 2018

It's resolved! Thanks for quick fix!

@mrgrauel
Copy link
Author

Resolved! Thanks

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

No branches or pull requests

3 participants