Student should start using more functions instead of just bringing the entire feature use case within the same function. For ex: Attempting database connect in every single function implementation is not a good practise.
Commit messages should be meaningful. Reviewers get insights into the changes made only through commit messages. It is one of the best practices in the industry.