First of all, welcome back! I think most developers create similar scripts that automate stuff - it's only natural since we are so lazy. ;) Some cool scripts there, but I've never heard of Makerlog. It looks pretty interesting, so I might check it out!
As for the code (I used this link to check the changes), here are some thoughts:
- The indentation and naming of variables are weird imo. Lining up all
=isn't necessary, and things aren't consistently camelCase, but I guess that's up to you. - Small inconsistencies like mixing of
letwhenconstcan be used. - Better to use
forEachinstead of a normalforloop, since you aren't actually counting anything and it's a lot more readable. - Maybe it's better to use a separate configuration file instead of having them as variables in the JavaScript file itself.
All in all, it could be pretty useful for some people! However, since programmers themselves probably already have something like this (the autovoter), I think the target audience is more than likely people who don't code, and so it could be more user friendly in my opinion. Anyway, as I said before, it's great to have you back, and I hope to see more contributions from you (and the rest of the PCSG guys) in the future!
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
Thank you for your review, @amosbastian! Keep up the good work!