-
Notifications
You must be signed in to change notification settings - Fork 0
Description
It's quite a long file (almost 200 lines), so you might want to separate the functionality into different files.
A few console.log and comments you could omit if you feel like it:
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L18
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L23
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L31
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L41
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L51
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L74
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L106 onwards
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L129
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L182 onwards
There's quite a bit of whitespace too, which is making it seem a lot longer than it really is.
Others:
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L48
requrlArray could be Camel cased.
Same with Qobj:
https://github.com/fcscripters/Devs-In-The-Detail/blob/master/index.js#L90