zoukankan      html  css  js  c++  java
  • 记录一次简单的PR过程(修补代码可读性)

    OpenCV使用了两步PR审核。当开发者提交pull request后,PR首先需要通过OpenCV Buildbot的自动测试,包括18种测试配置。
    当PR通过了自动测试后,OpenCV核心团队的成员将会对PR进行审核,若通过则此PR将会被集成进OpenCV主版本中。
    最近在翻看OpenCV的代码的时候,发现blobdetector.cpp下有两行注释比较碍眼,应该是明显的错误:
    由于OpenCV始终维护稳定的master,因此我们需要建议分支,分支可能为bug分支,或者为feature分支。

    这个操作可以直接在网站上完成,首先是将自己的版本更新到最新,而后建立分支
    注意我一般是用branch为3.4(很久之前是推荐这样的,但是现在我也有看到直接pr到master中的)

    在新建的分支中修改代码,并且保存,而其后在OpenCV中选择“Pull requests”
    其中画线的几个地方是特别需要注意到
    稍微等一下,出现这个黄圈,表示正在进行buildbot
    仔细看就可以发现,OpenCV实现不同平台加速的思想是:只写一份代码,则可在不同平台上高效运行。内部替用户实现了不同平台上的加速代码。


    全部成功后,进入review阶段(也可能不成功,也有人给你提意见)。较为简单的注释删除、bug修改通过的概率比较大;困难一点的是提交例子或者教程;最为复杂的是提交函数,需要你提供文档和Test。
    当reviewer关注到你的问题后,他们会给你提出很专业的建议,并且提供相关文档资料,应该说是非常正规的。比如这里:
    asmorkalov(来自intel)将我这个pr添加了photo的标签,并且委派给mshabunin(估计他是负责这个部分的),而后就会看到这个
    相关的修改就会被及时关注。当然如果别人提出的合理要求你老是不修改,也会被close.

    感谢阅读至此,希望有所帮助。






  • 相关阅读:
    更新处理函数在对话框的菜单中不能工作
    msn登录时80048820错误
    C#编程指南:使用属性
    sqlserver中''与null的区别
    IP地址与主机名识别问题
    给EXCEL表格奇偶行设置不同的背景颜色
    sqlserver2000发布订阅
    Excel数据导入到Sqlserver2000
    SQLSERVER 获取时间 Convert函数的应用
    对路径“C:\WINDOWS\Microsoft.NET\Framework\v1.1.4322\Temporary ASP.NET Files\aa\……”的访问被拒绝
  • 原文地址:https://www.cnblogs.com/jsxyhelu/p/13123633.html
Copyright © 2011-2022 走看看