Jon is a self-taught programmer, started in video games but now does web development. He follows principles, argues for scientific software development, and does not like writing in the 3rd person.
The team must understand all changes. Together we stand.
We pair to ensure shared code-knowledge and no “waiting for review” queues, and I advocate for isolated PRs so other devs can look at closed PRs and make sense of recent changes. It’s tempting to push unrelated changes into your ongoing PR to save time but it confuses every time.
By isolated I mean a PR should have a simple, understandable title, it’s changes should all relate to that, and the size should be small enough that no one sighs when they see it.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
The team must understand all changes. Together we stand.
We pair to ensure shared code-knowledge and no “waiting for review” queues, and I advocate for isolated PRs so other devs can look at closed PRs and make sense of recent changes. It’s tempting to push unrelated changes into your ongoing PR to save time but it confuses every time.
By isolated I mean a PR should have a simple, understandable title, it’s changes should all relate to that, and the size should be small enough that no one sighs when they see it.