Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Yes! Because the IDE cannot help in Github Pull Request reviews for example.


If a company did not run automatic tests and lints on whatever they felt was important to have when merging to production, to the fullest extent possible, I would stop everything and write those. This leaves reviewers free to focus on the intangibles: architecture, expressiveness, logic and data flow, and so on.


In my experience, Pull Requests aren't about catching syntax errors... the build will fail if there's errors like that. Rather, a Pull Request, and the code review involved, is about the underlying logic of the code; both with what it's doing it and how it's doing it.




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

Search: