News for hackers. Stuff that matters.

A comment on The Few, the Tired, the Open Source Coders. Parent comment

Re:The Few, the Tired, the Open Source Coders by throwawaywrench on Friday November 20, 2020 @12:10AM (#25157478)
>because they only solve the problem that the PR submitter needs solved right now.

This is something that can lead to angry interactions between maintainers and pull request raisers. Never mind adding technical debt, pull requests can outright break uses cases they don't care about in order to implement the single use case they do care about.

raiser: "Merge my PR. It fixes this issue."

maintainer: "It fixes this single issue but you've broken this for everyone else."

raiser: table flip

[ Parent | on HN ]
Re:The Few, the Tired, the Open Source Coders by sergeykish on Friday November 20, 2020 @01:06AM (#25157726)
Why? Just use own fork for a while. PR is sharing fix with others, it is visible even if not merged.
[ Parent | on HN ]
Re:The Few, the Tired, the Open Source Coders by rusticpenn on Friday November 20, 2020 @08:03AM (#25160045)
A good test strategy would fix the problem. "Your PR fails all these tests, so can you make them all work before submitting the request?"
[ Parent | on HN ]
Re:The Few, the Tired, the Open Source Coders by throwawaywrench on Monday November 23, 2020 @08:21PM (#25194138)
Definitely. You just need to keep an eye out for people removing failing tests because they think they are irrelevant :|
[ Parent | on HN ]