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.

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






  • 相关阅读:
    java基础(4)--javadoc文档与命令
    java基础(3)--pulic class与class的区别
    java基础(2)--main方法讲解
    java基础(1)--注释
    shell 测试文件状态运算符
    shell 算术运算符
    linux free命令详解
    shell 基本语法
    linux vim编辑器优化
    linux shell介绍
  • 原文地址:https://www.cnblogs.com/jsxyhelu/p/13123633.html
Copyright © 2011-2022 走看看