为什么复习很重要

欧宝体育平台怎么样社区评审对于Symfony框架的开发是必不可少的，因为在Symfony核心团队中评审、修复和合并这些ob娱乐下载请求和错误报告的人数远远超过了这些请求和错误报告的数量。

在ob娱乐下载Symfony问题跟踪器，你可以在商店里找到很多商品需要审查状态:

• 错误报告:需要检查Bug报告的完整性。是否遗漏了重要的信息?这个bug能被复制吗?
• 把请求: Pull请求包含修复bug或实现新功能的代码。对拉请求的审查确保它们被正确地实现，被测试用例覆盖，不会引入新的错误并保持向后兼容性。

请注意,任何对Symfony和PHP有基本了解的人都可以查看错误报告并提取请求ob娱乐下载．你不需要成为专家来提供帮助。

是建设性的

在你开始之前，请记住，你看到的是别人努力工作的结果。一个好的回顾评论感谢贡献者的工作，指出哪些地方做得好，哪些地方应该改进，并提出下一步的建议。

创建一个GitHub帐户

ob娱乐下载Symfony使用GitHub管理错误报告和提取请求。如果你想做评论，你就必须这么做创建一个GitHub帐户然后登录。

错误报告审查过程

开始检查的一个好方法是从需要审查的错误报告．

检讨的步骤如下:

1. 报告是否完整?

   方法创建的项目(“复制项目”)的链接ob娱乐下载Symfony框架或者是ob娱乐下载Symfony网站骨架这样就会产生bug。如果没有，报告至少应该包含足够的信息和代码示例来重现错误。

2. 重现Bug

   下载复制项目并测试该错误是否可以在您的系统上复制。如果报告者没有提供复制项目，则基于复制项目创建复制项目ob娱乐下载Symfony框架(或ob娱乐下载Symfony网站骨架）.

3. 更新问题状态

   最后，在错误报告中添加注释。感谢记者报告了这个漏洞．包括一行状态:<状态>在您的评论中触发我们卡森机器人更新问题的状态标签。您可以将状态设置为以下状态之一:

   需要工作如果这个bug不包含足够的信息以供复制，解释缺失的信息并将报告移至此状态。

   对我有用如果这个bug做包含足够的信息来复制，但在您的系统上工作，或者如果报告的错误是一个功能而不是一个错误，提供一个简短的解释并将报告移到这个状态。

   综述了如果可以重现错误，请将报告移到此状态。如果您创建了一个复制项目，请在注释中包含该项目的链接。

1 2 3 4 5

感谢@weaverryan创建这个错误报告!这看起来确实像一个bug。我在https://github.com/webmozart/some-project的“内核bug”分支中复制了这个bug。状态:审核

拉请求审查过程

检查拉取请求(pr)的过程与检查错误报告的过程类似。检查拉请求通常需要稍长时间，因为您需要了解已修复或添加的功能，并确定实现是否完整。

做部分评论是可以的!如果你只做了部分审查，评论一下你已经做了多少，然后让PR处于“需求审查”状态。

中选择一个拉请求需要检讨的pr遵循以下步骤:

1. PR完成了吗?

   每个拉请求都必须包含一个头，该头提供了关于PR的一些基本信息贡献的指导方针．

2. 基础分支正确吗?

   GitHub将PR所基于的分支显示在拉请求标题的下方。这个分支正确吗?

   • bug应该在包含bug的最古老的维护版本中修复。检查ob娱乐下载Symfony的发行时间表查找当前支持的最老版本。
   • 新特性总是应该添加到当前的开发版本中。检查ob娱乐下载Symfony的路线图查找当前开发版本。

3. 重现问题

   读取拉请求应该修复的问题。方法创建的新项目上重现该问题ob娱乐下载Symfony框架(或ob娱乐下载Symfony网站骨架)，并试着理解它存在的原因。如果链接的问题已经包含这样的项目，请安装它并在系统上运行它。

4. 检讨守则

   阅读拉请求的代码，并根据一些常见的标准检查它:

   • 代码是否解决了PR想要修复/实现的问题?
   • PR是否在处理范围内只有这个问题吗?
   • PR是否包含自动化测试?这些测试是否涵盖了所有相关的边缘情况?
   • PR是否包含足够的注释来理解它的代码?
   • 代码是否破坏了向后兼容性?如果是，PR头是否这么说?
   • PR是否包含弃用?如果是，PR头是否这么说?代码是否包含trigger_deprecation ()所有已弃用功能的语句?
   • 是否所有的弃用和向后兼容性中断都记录在最新的UPGRADE-X.X。md文件?这些解释是否包含“之前”/“之后”的例子，并有明确的升级说明?

   请注意

   最终，其中一些方面将被自动检查。

5. 测试代码

   从步骤3中提取项目，测试PR是否正常工作。中替换Symfony项目ob娱乐下载供应商通过运行以下Git命令，将PR中的代码替换为目录。插入PR ID(这是后面的数字＃在公关标题中)<标识>占位符:

   1 2 3

   $ CD vendor/ob娱乐下载symfony/symfony $ git获取原点拉/
                       /负责人:公关
                        $ git结帐pr
                       
                      

   例如:

   1 2

   $ git fetch origin pull/15723/head:pr15723 $ git checkout pr15723

   现在你可以测试项目与PR中的代码相对应。

6. 更新PR状态

   最后，在PR中添加评论。感谢贡献者为PR所做的工作．包括一行状态:<状态>在您的评论中触发我们卡森机器人更新问题的状态标签。您可以将状态设置为以下状态之一:

   需要工作如果PR还没有准备好合并，请解释您发现的问题并将其移到此状态。

   综述了如果PR满足上述所有检查，则将其移到此状态。核心贡献者将很快查看PR，并决定是否可以合并或需要进一步的工作。

1 2 3 4 5

谢谢你的工作@weaverryan !似乎您的测试用例没有覆盖计数器为零或更小的用例。你能补充一些测试吗?现状:需要工作