git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Documentation/CommunityGuidelines

From
Ramkumar Ramachandra <artagnon@gmail.com>
Date
Jun 12, 2013, 11:26 UTC
Message-ID
<CALkWK0==egyiSsndy-JC9a-N2pw4j5Q7sg2mm1gUFq6hqxjWvg@mail.gmail.com>
In-Reply-To
<20130611195452.GO22905@serenity.lan>
John Keeping wrote:
Show 21 quoted lines
> On Wed, Jun 12, 2013 at 12:16:28AM +0530, Ramkumar Ramachandra wrote:
>> John Keeping wrote:
>> >     Ugh, why this roundabout-passive-past tone?  Use imperative tone
>> >     like this:
>> >
>> >         ...
>> >
>> > vs.
>> >
>> >     We normally use the imperative in commit messages, perhaps like
>> >     this?
>> >
>> >         ...
>> >
>> > As my mother would say, "politeness costs nothing" ;-)
>>
>> The review is being honest about her feelings in the first one, and
>> being artificially diplomatic in the second one.
>
> I don't think it is artificially diplomatic, it's an attempt to convey a
> helpful tone in an email.

Okay, so answer this: Why did the reviewer deliberately use the "unhelpful" tone? Was she trying to attack the new contributor, and intend to harm the community? Or did she just say what came to her mind?

> As has been said elsewhere, it is easy to
> read an email in the wrong tone (there is an oft-cited statistic about
> the percentage of communication that is non-verbal, and which cannot be
> inferred from written text).
Yes, it is.
> For this reason I think it is important
> for reviewers to make an effort to minimise the risk that what they
> write can be interpreted as being aggressive.
Correct.
Show 8 quoted lines
>> Either way, I'm not interested in problems that have no solutions.
>> The only "solution" I see here is to suffocate every contributor until
>> they are "tactful enough" for the majority's liking, and "remove" the
>> ones that don't conform.  If you do have an alternate solution, please
>> share it with us.
>
> I don't have a solution, only a hope that regular contributors will
> learn from others how they can phrase review comments less aggressively.
The reviewer is not a thick-skinned bull that wants to harm the project.
4. Lead by example.  If you do not like how someone presents
themselves on the list, you counter it by presenting yourself nicely
on the list.  Others will follow your example, making that person's
behavior the minority.  It is far more powerful than explicitly
stating what is "acceptable" behavior and what is not.
Show 6 quoted lines
> I expect different people will read the same statement differently;
> people are from different cultures and what is considered acceptable in
> one culture can be considered rude in another.  We should aim to
> cultivate our own culture where we try to minimise the risk that what we
> write will be misinterpreted by someone with a different cultural
> background.

So you have agreed that "tone" is subjective, and that attempting to objectively state the "right tone" is a lost cause.

The solution to the problem, as I have already explained several times is to:
- Define an objective basis for people to react.
- Lead by example, and influence other contributors to follow your style.
What everyone is doing differently:
- Taking offense at every possible juncture.
- Taking sides and voting.  Ganging up and playing politics.
- Making bad irrational arguments in the "right tone".
- Invalidating entire arguments, on the basis of tone.
- Making tone the entire subject of discussion, ignoring content.
- Bringing "majority opinion" to a rational argument.
Previous: John KeepingNext: John Keeping
Message 41 of 63 in “Documentation/CommunityGuidelines”
  1. Documentation/CommunityGuidelinesRamkumar Ramachandra, Jun 10, 2013
  2. Célestin MatteJun 10, 2013
  3. Matthieu MoyJun 10, 2013
  4. Robin H. JohnsonJun 10, 2013
  5. Junio C HamanoJun 10, 2013
  6. Jonathan NiederJun 10, 2013
  7. Ramkumar RamachandraJun 10, 2013
  8. A Large Angry SCMJun 10, 2013
  9. Ramkumar RamachandraJun 10, 2013
  10. A Large Angry SCMJun 10, 2013
  11. Felipe ContrerasJun 11, 2013
  12. Ramkumar RamachandraJun 11, 2013
  13. Michael HaggertyJun 11, 2013
  14. Felipe ContrerasJun 11, 2013
  15. Ramkumar RamachandraJun 11, 2013
  16. Felipe ContrerasJun 11, 2013
  17. Thomas RastJun 11, 2013
  18. Ramkumar RamachandraJun 11, 2013
  19. Michael HaggertyJun 11, 2013
  20. Felipe ContrerasJun 11, 2013
  21. Ramkumar RamachandraJun 11, 2013
  22. Michael HaggertyJun 11, 2013
  23. Ramkumar RamachandraJun 11, 2013
  24. Junio C HamanoJun 11, 2013
  25. Felipe ContrerasJun 11, 2013
  26. Felipe ContrerasJun 11, 2013
  27. Brandon CaseyJun 11, 2013
  28. Theodore Ts'oJun 12, 2013
  29. Ramkumar RamachandraJun 12, 2013
  30. Felipe ContrerasJun 12, 2013
  31. Felipe ContrerasJun 11, 2013
  32. Thomas RastJun 11, 2013
  33. Felipe ContrerasJun 11, 2013
  34. Thomas RastJun 11, 2013
  35. Felipe ContrerasJun 11, 2013
  36. Junio C HamanoJun 11, 2013
  37. Michael HaggertyJun 11, 2013
  38. John KeepingJun 11, 2013
  39. Ramkumar RamachandraJun 11, 2013
  40. John KeepingJun 11, 2013
  41. Ramkumar RamachandraJun 12, 2013
  42. John KeepingJun 12, 2013
  43. Michael HaggertyJun 11, 2013
  44. John KeepingJun 11, 2013
  45. Philip OakleyJun 11, 2013
  46. John SzakmeisterJun 12, 2013
  47. Jakub NarebskiJun 12, 2013
  48. Philip OakleyJun 12, 2013
  49. Felipe ContrerasJun 11, 2013
  50. Jeff KingJun 11, 2013
  51. Junio C HamanoJun 11, 2013
  52. Felipe ContrerasJun 11, 2013
  53. Theodore Ts'oJun 12, 2013
  54. Felipe ContrerasJun 12, 2013
  55. Ramkumar RamachandraJun 12, 2013
  56. Junio C HamanoJun 12, 2013
  57. Michael HaggertyJun 13, 2013
  58. Junio C HamanoJun 13, 2013
  59. Felipe ContrerasJun 11, 2013
  60. Ramkumar RamachandraJun 11, 2013
  61. Thomas AdamJun 13, 2013
  62. Felipe ContrerasJun 13, 2013
  63. Christian CouderJun 14, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.