记一次向reacor-netty提交代码及对review工作的思考

如何获取贡献的点

  • stackoverflow 网站reactor-nettyreactor-core 标签下的问题,看到有人提问而且问题也在issue列表中,向owner申请解决问题,owner同意后会将对应的issue标记为 help wanted
  • issue列表,可以从issue列表中选择自己可以处理的issue。
  • 阅读文档时发现错误或这发现代码实现和文档不符合,可以提交issue并创建一个PR。
  • 实现代码中标记为todo的功能。

如何让PR顺利merge

  • 首先要向项目的owner认领issue,避免多个人单独解决issue

  • 增加一个新功能:

    • 详细描述新功能/理解他人提出的功能增强需求
    • 设计功能实现方案
    • 和owner一起讨论方案(讨论,修改,讨论......)
    • 编码(功能代码和测试代码)
    • 测试
    • 提交PR等待更多的人review本次PR
  • 修复bug

    • 复现bug(bug不能复现我们也很难解决)
    • 详细描述bug产生的原因及影响
    • 制定bug修复方案
    • 和owner一起讨论bug修复方案(讨论,修改,讨论......)
    • 编码解决(bug修复代码和测试代码)
    • 测试
    • 提交PR等待更多人review本次PR
  • 修复文档错误

    • 描述文档错误或文档和实现不一致的内容
    • 修改文档
    • 提交PR等待更多人review本次PR
  • 处理review意见

    • 理解review意见
    • 和提出人确认review意见及下一步修改方案
    • 重新提交代码或文档

PR 顺利合并的关键是:沟通,沟通,沟通。

困难

个人认为最大的困难是用英语沟通,尤其是口语化的英语更难准确理解他人要表达的意思,Google翻译已经无法满足诉求了。好一点的是,不管你的英语水平怎么样,他们都能理解你要表达的意思。

向开源社区review学习什么?

  • 让最懂的人review代码
  • owner或committer提前参与方案的讨论
  • 看护代码架构,保持代码架构的一致性,摒弃“又不是不能用的”的思想。
  • 看护代码的可扩展性
  • 测试用例也要认真review
  • 注释也是review的重点,保证注释的准确性至关重要。
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 205,236评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,867评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,715评论 0 340
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,899评论 1 278
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,895评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,733评论 1 283
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,085评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,722评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,025评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,696评论 2 323
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,816评论 1 333
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,447评论 4 322
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,057评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,009评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,254评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,204评论 2 352
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,561评论 2 343

推荐阅读更多精彩内容