835 B
835 B
Code Review Changes
https://docs.google.com/document/d/19d5U-ieQyGVVNQjj0rUouzIqF53evEP6yss-wyqNXiQ/edit?usp=sharing
| Comment | Adjustment |
|---|---|
The package com.danilafe.fencelessgrazing can be changed to something more relevant. |
Package name follows standard conventions of reverse domain name. No changes made. |
| (General remark) commenting is sparse. | Every function and field in the codebase documented using KDoc; documentation intermittently updated on student website. |
| No unit tests | Not worth it to do UI tests; cow description language developed for testing databases elsewhere. |
| Missing README. | README added with compilation instructions; gradle wrapper adjusted to work out of the box. |