You are viewing a single comment's thread from:

RE: LogoTurtle v0.0.15: Add GOTO and Configure Project CI on Travis

in #utopian-io6 years ago

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, but let 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.logs everywhere. Is this necessary?
  • Variables i, x and y - 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.

[utopian-moderator]

Sort:  

Thank you for your review, @amosbastian! Keep up the good work!