> On 13 January 2016 at 03:10, Tom Lane <tgl@xxxxxxxxxxxxx> wrote: > One thing to think about here is the idea of framing the process. One reason it might be a good idea to have a "respect the commons" clause is that it becomes a good way to think about the interaction of review and technical discussion. I.e. both sides want to improve > the software. The focus is on the software, not on the other person. > People *can* take offense when you say their code is not good enough, particularly when it is true, because for better or worse we do often identify with what we produce. But I would hope that if the focus is on improvement of the software the this becomes at least a > bit less of a problem..
> Best Wishes, > Chris Travers Very good point. I know personally I feel more hurt at my code being criticized than someone criticizing some random aspect of me. That said Perhaps something like "we judge contributions primarily based on how easily it fits into our existing code base and the popularity of the problem or feature it targets" I was going to talk about correctness and all that, but I think that's kind of inferred by the comment about fitting into our existing code base. If it's not correct it wouldn't fit anyway. Thanks, Regina |
