为什么复习是很重要的

欧宝下载链接社会评论Symfony框架的发展至关重要,因为有很多拉请求和bug报告比在Symfony核心团队成员审查,欧宝娱乐app下载地址修复并将它们合并。

在欧宝娱乐app下载地址Symfony问题跟踪器,你可以找到许多项需要审查状态:

• 错误报告:Bug报告需要检查完整性。任何重要的信息缺失?这个错误可以复制吗?
• 把请求:拉请求包含代码,修复一个缺陷或实现新的功能。把请求的审查确保正确实现它们,被测试用例,覆盖不引入新的缺陷,保持向后兼容性。

请注意,一些基本的人都熟悉Symfony和PHP可以回顾bug报告和拉请求欧宝娱乐app下载地址。你不需要一个专家的帮助。

是建设性的

在您开始之前,请记住,你们是看别人的努力工作的结果。贡献者的好评论谢谢他们的工作,确定是什么做得好,确定下一步应该改进和建议。

创建一个GitHub帐户

欧宝娱乐app下载地址Symfony使用GitHub管理错误报告和请求。如果你想做评论,你需要创建一个GitHub帐户并登录。

错误报告的审查过程

开始复习的好方法就是选择一个错误的报告bug报告需要审查。

审查的步骤是:

1. 报告完成吗?

   良好的bug报告包含一个链接到一个项目(“繁殖项目”)的创建欧宝娱乐app下载地址Symfony框架或者是欧宝娱乐app下载地址Symfony的网站框架复制错误。如果它不,报告至少应包含足够的信息和重现这个bug的代码示例。

2. 复制错误

   下载复制项目和测试缺陷是否可以复制在您的系统上。如果记者不提供复制项目,创建一个基于一个欧宝娱乐app下载地址Symfony框架(或欧宝娱乐app下载地址Symfony的网站框架)。

3. 更新这个问题的状态

   最后,将评论添加到错误报告。谢谢记者报告错误。包括线状态:<状态>在你的评论引发我们的卡森机器人更新的状态标签的问题。你可以将状态设置为下列之一:

   需要工作如果错误不复制包含足够的信息,解释信息缺失和移动这个状态报告。

   适合我如果错误做包含足够的信息被复制,但在您的系统上工作,或者如果错误是一种特性,而不是bug报告,提供一个简短的解释,这种状态报告。

   综述了如果你能复制错误,把这种状态报告。如果您创建了一个复制的项目,包括项目的链接在你的评论。

1 2 3 4 5

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

把请求审查过程

复习的过程拉请求(PRs)类似于一个用于错误报告。评论拉请求通常需要一段时间,因为你需要理解的功能固定或添加,找出实现是否完成。

这部分是可以评论!如果你做部分评论,评论你有多远,把公关在“需求评估”状态。

选择一个请求PRs需要审查遵循以下步骤:

1. 公关是完整的吗?

   每个拉请求必须包含一个头,让一些基本的关于公关的信息。你可以找到模板的标题中贡献的指导方针。

2. 基地分支正确吗?

   GitHub显示分支,公关是基于下面的标题将请求。分支是正确的吗?

   • 错误应该固定在最古老、维护版本包含错误。检查欧宝娱乐app下载地址Symfony的发布时间表目前发现的最古老的支持版本。
   • 新特性应该被添加到当前的开发版本。检查欧宝娱乐app下载地址Symfony的路线图找到当前的开发版本。

3. 重现这个问题

   读拉请求应该解决的问题。重现这个问题在创建一个新项目欧宝娱乐app下载地址Symfony框架(或欧宝娱乐app下载地址Symfony的网站框架)并试图理解它存在的原因。如果连接问题已经包含这样一个项目,在您的系统上安装并运行它。

4. 检查代码

   读拉请求的代码并检查它对一些常见的标准:

   • 代码解决这个问题的公关是为了修复/实现吗?
   • 公关留在范围地址吗只有这个问题吗?
   • 公关包含自动化测试吗?这些测试覆盖所有相关的边界情况吗?
   • 公关是否包含足够的评论来了解其代码?
   • 打破向后兼容性的代码吗?如果是的,公关头是这么说的吗?
   • 公关包含的用法吗?如果是的,公关头是这么说的吗?代码是否包含trigger_deprecation ()声明所有弃用功能吗?
   • 都不支持和打破记录的最新UPGRADE-X.X的向后兼容性。md文件?做这些解释包含”之前,“/”后“有明确的升级说明例子吗?

   请注意

   最终,这些方面将被自动检查。

5. 测试代码

   把你的项目从步骤3和测试是否公关工作正常。取代Symfony项目欧宝娱乐app下载地址供应商目录由公关中的代码运行以下Git命令。插入的公关ID(后面的数字#在公关标题)<标识>占位符:

   1 2 3

   cd供应商/ symfo欧宝娱乐app下载地址ny / symfony git获取美元拉/起源
                       /负责人:公关
                        美元git checkout公关
                       
                      

   例如:

   1 2

   git获取来源把美元/ 15723 /负责人:pr15723 git checkout pr15723美元

   现在你可以测试的项目对公关中的代码。

6. 更新PR状态

   最后,将评论添加到公关。感谢贡献者的公关工作。包括线状态:<状态>在你的评论引发我们的卡森机器人更新的状态标签的问题。你可以将状态设置为下列之一:

   需要工作如果公关尚未准备好被合并,解释你所发现的问题和移动到这个状态。

   综述了如果公关满足所有上面的检查中,移动到这个状态。核心贡献者将很快看公关和决定是否可以合并或需要进一步的工作。

1 2 3 4 5

谢谢你@weaverryan研究这个!看来你的测试用例不覆盖情况下当计数器是零或更小。你能添加一些测试吗?状态:需要工作