语调

我们不要求你写得非常正式，甚至不要求你写得没有错误。记住这一点:不要骂人，要尊重他人。

不要用愤怒或咄咄逼人的语气回答。你很生气，我们理解，但咒骂和辱骂并不能真正鼓励任何人帮助你。深呼吸，数到10，然后试着很明显解释你遇到的问题。

包容性的语言

为了包容更广泛的人群，建议使用不表示特定性别的人称代词。除非有人说了他们的代词，否则就用“他们”，“他们”来代替“他”，“她”，“他的”，“她的”，“他/她的”，“他/她的”等等。

尽量避免使用可能被认为是排斥、不必要的性别(例如以男性或女性为基础的词语)、种族动机或针对社会中的特定群体的措辞。例如，建议使用“乡亲们”、“团队”、“每个人”等词，而不是“伙计们”、“女士们”、“美国佬”等。

给予积极的反馈

在审查问题和请求时，你可能会遇到一些建议(包括补丁)，这些建议不能反映你的想法，不好，或者完全错误。

现在，当你准备你的评论时，考虑一下作者在他们的想法上花费了多少精力和时间，以及你的回答会给他们什么感觉。

你是否正确理解了他们的意图?或者你只是在做假设?无论你的回答是什么，都要明确。记住，人们并不总是理解你在网上的意图。

避免使用可能被视为涉及个人特征的术语(“愚蠢”，“愚蠢”)。假设每个人都是聪明和善良的。

即使有些事情是绝对错误或“坏主意”，保持尊重，不要陷入无休止的“你错了”的讨论或“火焰战”。

不要使用夸张的词(“总是”，“从不”，“无休止”，“什么都没有”，“最糟糕的”，“可怕的”，“可怕的”)。

别:"我不喜欢你写代码的方式"-没有明确的解释为什么你不喜欢它的写作方式。

好:“我发现这段代码很难读，因为有很多嵌套的if语句，你能让它更容易读吗?”通过封装一些细节，或者添加一些注释来解释整体逻辑。”-你解释一下为什么你觉得代码很难读而且提出一些改进建议。

如果一段代码实际上是错误的，请解释原因:

• 这段代码不符合Symfony的CS规则。欧宝娱乐app下载地址请参阅[…]]浏览详情．
• 欧宝娱乐app下载地址Symfony 3仍然使用PHP 5，并且不允许使用标量类型提示。．
• 我认为现在的代码可读性降低了这里要小心，一定要解释为什么你认为代码可读性较差，也许会给出一些建议?

拒绝的正当理由的例子:

• 我们在过去尝试过(链接到相关的PR)，但由于XXX原因我们需要恢复它。
• 在合并Symfony分支时，该更改将引入太多的合并冲突。欧宝娱乐app下载地址在过去，我们总是拒绝这样的改变。
• 我对这一变化进行了分析，它极大地损害了性能(如果您不进行分析，这只是一种意见，所以我们可以忽略)
• 代码不符合Symfony的CS规则(例欧宝娱乐app下载地址如使用［］而不是数组()）
• 我们只提供与非常流行的项目的集成(例如，我们集成Bootstrap，但不集成您自己的CSS框架)
• 这将需要为一个看起来不那么重要的功能添加大量代码并进行大量更改。这可能会影响未来的维护。

要求改变

很少有东西从一开始就是完美的，而代码本身是好的。它可能不是最优的，也可能不符合Symfony编码风格。欧宝娱乐app下载地址

同样，要明白作者已经在这个问题上花了时间，要求(小的)修改可能会被误解或被视为人身攻击。

感谢他们的工作(到目前为止)，保持积极的态度，真正帮助他们做出伟大的贡献。特别是如果他们是第一次投稿。

用“请”、“谢谢”和“你能”这样的词来代替提出要求;

• “谢谢你到目前为止所做的工作。我留下了一些改进建议，以使代码更具可读性。”
• “你的代码包含一些编码风格的问题，你能在我们合并之前解决这些问题吗?”谢谢你”
• “请使用4个空格而不是制表符”，“这需要在上一行”;

在拉请求审查期间，你通常可以留下多条评论，你不必一直使用“请”。但这也无伤大雅。

这可能看起来没什么，但说“谢谢”确实会让别人感觉更受欢迎。

使用幽默

简而言之:极端的不当行为是不被容忍的，甚至可能会让你被封号;保持真诚和友好。

不要在严肃的话题上使用讽刺，那不属于Symfony社区。欧宝下载链接欧宝娱乐app下载地址不要忽视别人的问题;我猜这是不应该发生的?．

即使有人的解释是“让人开玩笑”，这对他们来说也是一个真正的问题。拿这件事开玩笑并不能帮助解决他们的问题，只会让他们更加烦恼觉得自己愚蠢．相反，试着去发现问题的真正原因。

最后的话

如果你“没有”遵循这些建议，也不要感到难过。只要你的意图是好的，你没有冒犯或侮辱任何人;你可以解释你误解了他，你不是有意要边缘化他，或者只是失败了。

但如果你的道歉不是真心的，不要说“只是因为”将失去其他开发者的信任和尊重。

己所不欲，勿施于人。