AuthMe Reloaded

AuthMe Reloaded

3M Downloads

TODO comments

CoolKangaroo2022 opened this issue ยท 0 comments

commented

Hello! I find in your Coding Styleguide that

Similarly, create issues in the issue tracker for larger problems or improvements instead of just writing a TODO comment. TODO comments are guaranteed to get forgotten if they are not actively tended to.

However, I still witness some TODO comments, e.g., a todo comment in IndividualFilesPersistenceHandlerTest.java in 2016.

I also find TODO comments in the following files:

  • src/main/java/fr/xephi/authme/command/executable/authme/AccountsCommand.java
  • src/main/java/fr/xephi/authme/datasource/converter/RakamakConverter.java
  • src/main/java/fr/xephi/authme/listener/EntityListener.java
  • src/main/java/fr/xephi/authme/task/purge/PurgeService.java
  • src/main/java/fr/xephi/authme/datasource/converter/XAuthConverter.java
    ......

Would you like to remove these TODO comments, or move them to issues, or just leave them alone?
If there is anything I can help with?