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

Don’t. I would refuse to review a PR with 9000 LOC and 63 new files even if written by a human. Something that large needs to be discussed first to agree on an architecture and general approach, then split in manageable pieces and merged piece-wise in a feature branch, with each individual PR having reasonable test coverage, and finally the feature branch merged into master.


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

Search: