Hacker News new | past | comments | ask | show | jobs | submit login

Depending on the team you can get away with some of these. Have a manager/PHB that actively discourages unit testing? You don't need to commit your unit tests or write them at all! No one fixes an instance of copy/paste coding? Great, now you too can be efficient by copying and pasting everywhere. No one uses a code formatter or uses different formatting styles? Good! Now your style can shine!

These are all good red flags to check for in existing code bases before you join a team. If you see them, chances are their code review process is poor or non-existent and you'll be fighting a battle against crap code every single day.




Looking through the codebase for these to assess the teams review health is a fantastic idea!




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: