TLDR: Thinking this is a real PR will only hurt you. Just list the things you know I approached this PR with warmth. As I thought that's what they wanted when they repeated this PR is from a new dev. You don't have time to address all the problems in this PR. The time I spent in providing articles, best practices, etc where a waste of time and ended up hurting my assessment. I wasted time downloading the repo and running the code to confirm methods didn't exist. My feedback from Woven was a literal bullet point of "good that you mentioned this" and "not good that you didn't mention this". When filling this out just do basic statements. You're just checking boxes that "people" can easily scan and ensure you have knowledge of security, scalability, testing, databasing, etc.