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

If there’s anything I would want to run to verify, I ask the author to add a unit test. Generally, the existing CI test + new tests in the PR having run successfully is enough. I might pull and run it if I am not sure whether a particular edge case is handled.

Reviewers wanting to pull and run many PRs makes me think your automated tests need improvement.





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

Search: