Skip to content

Conversation

@nanoticity
Copy link

No description provided.

@jamespeilunli
Copy link
Collaborator

jamespeilunli commented Aug 10, 2025

Btw when I said "delete comments that say TODO if you have completed them" I meant like the whole comment (like everything in the line after the //, not just "TODO"), but it doesn't really matter in this case so I still approved it

The reason you should delete these lines is that to TODO and what it's asking you to do is redundant because your code that addresses the todo is self explanatory. If you needed to explain it, you would add a new comment that explained what your code does

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants