Use fs.Dirent objects, and extract metadata in the background about video-hub-app

[whyboris]

A better way to see which node version is in which Electron version:
https://github.com/electron/releases

Looking at how the v3.0.0 was released, once they are done with beta versions, they have a few nightly versions and then release it for good. Feels like we'll get v4.0.0 around New Year's 🎄

[whyboris]

When working on this, should be easy to close this issue too: #24 👍

[cal2195]

I just tried beta 11 there on master and it works! 🎉

[whyboris]

Electron v4.0 is out as of 1 hour ago 🎉
https://github.com/electron/electron/releases/tag/v4.0.0
We'll upgrade in a few days 👍 -- or feel free to open up this change as a PR with the version bump 💯

[cal2195]

A wild PR has appeared! 🍃 #33

[whyboris]

Thank you!

I'm happy to update to node v10 and to this new approach -- will do so along with my angular7 branch 👍

ps -- some users have requested we ignore files that start with . and with _ which we'll do as well 👌

[whyboris]

master has been updated with node 10.14 so this is ready for a PR -- whenever you'd like 🥇

[cal2195]

Darn, looks like electron 3.0.10 only has node v10.2.0 - https://electronjs.org

Unless I'm wrong, this will have to wait until electron 4?

[whyboris]

I think you're right - https://electronjs.org/releases#3.0.0 electron 3 comes with Node v10.2.0
I suspect electron 4 is coming out soon - before we have our next release 👌

[cal2195]

Nice! 🎉

I noticed you tried to use v4 in your original upgrade - was there any problems, and would it be possible to have a beta branch to test it out?

[whyboris]

It's worth a try -- feel free to give it a go: electron 4 beta 10 came out 30 minutes ago:
https://github.com/electron/electron/releases/tag/v4.0.0-beta.10
I tried it before but something wasn't working (could have been something else), so I just rolled back to decrease the number of moving pieces, fixed the problem, and never tried with the version 4 beta 🐟

edit: tried the beta 10 and the app wouldn't compile:

internal/validators.js:125
    throw new ERR_INVALID_ARG_TYPE(name, 'string', value);
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "file" argument must be of type string. Received type object
    at validateString (internal/validators.js:125:11)

I'm happy to just wait for the non-beta release 👌