尊重审核评论

编辑该页面

警告:你浏览的文档欧宝官网下载appob娱乐下载Symfony 5.1,不再维护。

这个页面的更新版本Symfob娱乐下载ony 6.2(当前的稳定版本)。

尊重审核评论

审查问题和请求是一个伟大的方式开始使用Symfony的社区做出贡献。欧宝体育平台怎么样ob娱乐下载任何人都可以做到!但是在你给的评论,退一步想,是你说你要表达什么?

交流在互联网上除了文本可以构成一个巨大的挑战,特别是如果你记住Symfony社区是全球范围内,由各种各样的人有不同的想法和观点。欧宝体育平台怎么样ob娱乐下载

不是每个人都讲英语或能够使用一个键盘。有些人可能会有阅读障碍或类似情况影响他们写作。

更不用说,一些可能有一个糟糕的经历从先前的贡献(其他项目)。

你不是一个人。本指南将帮助你写建设性,尊重和有用的评论和回复。

提示

本指南不讲课你“符合”或—你的想法和意见但帮助你更好的沟通,防止可能的混淆,并保持Symfony社区一个欢迎的地方。欧宝体育平台怎么样ob娱乐下载你可以不同意别人的意见,但不要失礼。

首先,接受许多编程决策意见。你喜欢讨论权衡,并迅速达成决议。这不是关于对或错,但使用作品。

的语气

我们不希望你完全正式的、或甚至无错的英文写作。记住:不要发誓,尊重他人。

不要在愤怒或回复与积极的基调。如果你生气,我们明白,但是发誓/诅咒和骂人不鼓励任何人帮助你。深吸一口气,数到10,尝试很明显解释一下你遇到的问题。

包容性的语言

以包容广泛的人群,建议使用人称代词,不建议一个特定的性别。除非有人说他们的代词,使用“他们”、“他们”,而不是“他”、“她”、“他”、“她”、“他/她”,“他/她”,等等。

尽量避免使用可能被认为是不含的措辞,不必要的性别(例如,男性或女性基础单词,),种族动机或挑出一个特定的社会团体。例如,推荐使用单词像“人”、“团队”,“每个人”,而不是“人”,“女士”,“美国佬”,等等。

给予积极的反馈

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

现在,当你准备你的评论,作者考虑的工作量和时间花在他们的想法和你的反应会使他们感觉如何。

你正确理解他们的意图吗?或者你在假设吗?不管你的反应,是明确的。记得网上人们并不总是理解你的意图。

避免使用术语可以被视为指个人特质(“笨”、“愚蠢”)。假设每个人都是聪明的和善意的。

提示

好的问题避免判断和避免假设作者的观点。

也许你可以要求澄清?建议另一个吗?或者提供一个简单的解释为什么你不同意他们的建议。

  • 这看起来是错误的。你确定它是正确的吗?(如输入错误或语法错误)
  • 你觉得“RequestFactory”而不是RequestCreator ?

即使一些真的错了或“一个坏主意”,保持尊重,不要陷入无休止的讨论你错了或“火焰战争”。

不要使用夸张(“总是”、“从不”、“没完没了地”,“没什么”,“坏”,“可怕的”、“可怕的”)。

别:“我不喜欢你写的这段代码”——没有明确解释为什么你不喜欢如何写。

好:“我很难读这段代码有很多嵌套的if语句,你可以让它更可读吗?通过封装一些细节或者添加一些注释来解释整体的逻辑。”——你解释为什么你觉得难读的代码给出一些改进的建议。

如果一段代码实际上是错误的,原因:

  • “这段代码不符合Symfony的CS规则。ob娱乐下载请参阅[…详情。”
  • “ob娱乐下载Symfony 3仍然使用PHP 5和不允许使用标量类型提示。”
  • “我认为现在降低代码的可读性。”- careful here, be sure explain why you think the code is less readable, and maybe give some suggestions?

有效的原因拒绝的例子:

  • “我们试过在过去(链接到相关的公关),但我们需要恢复它XXX原因。”
  • “改变将引入太多的合并冲突当合并Symfony分支。ob娱乐下载在过去我们一直拒绝改变这样的。”
  • “我描述这一变化和疼性能显著”——如果你不配置文件,这是一个观点,所以我们可以忽略
  • “代码不匹配Symfony的CS规则(例ob娱乐下载如使用[]而不是数组())”
  • “我们只提供集成非常流行的项目(如积分引导但不是自己的CSS框架)”
  • “这需要添加大量的代码和做很多改变看起来不那么重要的一个特性。在未来,可能影响维护。”

要求更改

从一开始就很少的东西是完美的,而代码本身是好的。它可能不是最优或符合Symfony的编码风格。ob娱乐下载

再次,理解作者在这个问题上已经花了时间和要求(小)变化可能被误解或视为人身攻击。

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

使用诸如“请”、“谢谢”和“能”的要求;

  • “谢谢你的工作。我离开了一些建议改进使代码更易于阅读。”
  • “代码包含一些编码风格的问题,你能解决这些在我们合并?谢谢你”
  • “请使用4个空格代替标签”,“这就需要在前一行”;

在拉请求审查通常可以留下更多然后一个评论,你不必使用“请”。但它不会伤害。

好像不是很多,但是说“谢谢”并让别人感到更受欢迎。

防止升级

有时当人们接收反馈他们可能会防守。在这种情况下,最好是尝试以不同的方式方法的讨论,不进一步升级。

如果你想要有人来调解,请加入#普通发布版频道ob娱乐下载Symfony松弛有一个安全的环境,保持共同的目标一起工作。

使用幽默

简而言之:极端的不当行为将不会被容忍,甚至可能让你禁止;保持真实和友好。

不要嘲讽一个严肃的话题,不是属于Symfony社区。欧宝体育平台怎么样ob娱乐下载不要忽视别人的问题;我想这是不应该发生的?

即使一个人的解释是“邀请开玩笑”,这是一个真正的问题。开玩笑关于这无助于解决问题,只会让他们觉得自己愚蠢。而不是试图发现问题是什么。

最后的话

不要难过,如果你遵循这些提示“失败”。只要你的意图是好的,你没有冒犯或侮辱任何人;你能解释你误解,你不是故意排斥或者仅仅是失败了。

但不要说,“因为”,如果你并不意味着你道歉从其他开发人员那里失去信誉和尊重。

己所不欲做给你们。

这项工作,包括代码示例,许可下Creative Commons冲锋队3.0许可证。