Note: Based on the "collate/" in the master branch on retrieved on Sunday.
main:
Note: fxml ignored. Also, mostly UI code, which may lack certain coding aspect (e.g. assertion / logging may not be very meaningful in UI code). Try to write code for other component if possible.
Comment is generally well written, but may be excessive in some places, i.e. if the code is very clear, you may not need to comment on it.
SLAP can be improved in some places. e.g. in TaskListCard(ReadOnlyTask task, int displayedIndex).
test:
The naming for the tests does not follow the suggested three-parts naming scheme.
The comment in the tests may be too excessive. :-)
There are some duplicated code. Consider using additional methods or setup() / teardown().
Note: Based on the "collate/" in the master branch on retrieved on Sunday.
main:
Note: fxml ignored. Also, mostly UI code, which may lack certain coding aspect (e.g. assertion / logging may not be very meaningful in UI code). Try to write code for other component if possible.
Comment is generally well written, but may be excessive in some places, i.e. if the code is very clear, you may not need to comment on it.
SLAP can be improved in some places. e.g. in TaskListCard(ReadOnlyTask task, int displayedIndex).
test:
The naming for the tests does not follow the suggested three-parts naming scheme.
The comment in the tests may be too excessive. :-)
There are some duplicated code. Consider using additional methods or setup() / teardown().