You are viewing a single comment's thread from:
RE: LogoTurtle v0.0.15: Add GOTO and Configure Project CI on Travis
Thanks for the contribution, @justyy. It's cool to see you are continuing the development of this project after a 9 month hiatus! I'm curious, why did you decide to continue?
As for the code:
- There are still some places where
const
should be used, butlet
is used instead. - Indentation is 4 spaces, but in your other browser extension that I just reviewed you use 2 spaces. I guess it doesn't really matter that much, as long as it is consistent across the project itself.
- Loads of
console.log
s everywhere. Is this necessary? - Variables
i
,x
andy
- they could be more descriptive in my opinion.
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!