search

getstation / electron-process-manager

Process manager UI for Electron applications
189 stars 41 forks source link

Memoize misspelled #4

Closed zekenie closed 6 years ago

zekenie commented 6 years ago

const memoize = require("memoizee"); in src/ProcessStatsReporter.js

:)

alexstrat commented 6 years ago

Are you sure it's a mistake? If you you talking about "memoizee", it's actually a real package that i'm using => https://github.com/medikoo/memoizee

zekenie commented 6 years ago

Ah, good question. I thought it was a mistake because when I ran my app after installing electron-process-manager with yarn, it told me that memoizee was missing. It looked like a typo so I figured it was... not sure what's going on in that case

alexstrat commented 6 years ago

I may have forgotten a dependency.

Can you try with npm install electron-process-manager@0.3.0 ?

guillaumearm commented 6 years ago

electron-process-manager@0.3 work fine, and it's not a problem in newest version because we don't use memoizee anymore.

@zekenie : is your issue is solved ? if not, can you paste your package.json here ? maybe you have dependencies issues on your project.

zekenie commented 6 years ago

I think this is solved. I don't remember what exactly was going on