Overall the project is overengineered since it uses the diamond pattern although it is unnecessary. The team, however, decided to use the diamond pattern intentionally to show how it could be used. Although we have not found any significant issues with the code itself, we very highly recommend improving the test suite both for: 1) the user-facing functionality of the project, and 2) the diamond pattern itself. We also provide a few recommendations for improving the code. **Update:** the report has been revised based on commit `5978a3d`. Notably, the team improved the test suite.
Low | Medium | High | Critical | Total | |
---|---|---|---|---|---|
Not fixed | 3 | - | 1 | - | 4 |
Fixed | 3 | - | - | - | 3 |
Total | 6 | 0 | 1 | 0 | 7 |