記一次向reacor-netty提交代碼及對(duì)review工作的思考

如何獲取貢獻(xiàn)的點(diǎn)

  • stackoverflow 網(wǎng)站reactor-nettyreactor-core 標(biāo)簽下的問題,看到有人提問而且問題也在issue列表中,向owner申請(qǐng)解決問題,owner同意后會(huì)將對(duì)應(yīng)的issue標(biāo)記為 help wanted
  • issue列表,可以從issue列表中選擇自己可以處理的issue。
  • 閱讀文檔時(shí)發(fā)現(xiàn)錯(cuò)誤或這發(fā)現(xiàn)代碼實(shí)現(xiàn)和文檔不符合,可以提交issue并創(chuàng)建一個(gè)PR。
  • 實(shí)現(xiàn)代碼中標(biāo)記為todo的功能。

如何讓PR順利merge

  • 首先要向項(xiàng)目的owner認(rèn)領(lǐng)issue,避免多個(gè)人單獨(dú)解決issue

  • 增加一個(gè)新功能:

    • 詳細(xì)描述新功能/理解他人提出的功能增強(qiáng)需求
    • 設(shè)計(jì)功能實(shí)現(xiàn)方案
    • 和owner一起討論方案(討論,修改,討論......)
    • 編碼(功能代碼和測(cè)試代碼)
    • 測(cè)試
    • 提交PR等待更多的人review本次PR
  • 修復(fù)bug

    • 復(fù)現(xiàn)bug(bug不能復(fù)現(xiàn)我們也很難解決)
    • 詳細(xì)描述bug產(chǎn)生的原因及影響
    • 制定bug修復(fù)方案
    • 和owner一起討論bug修復(fù)方案(討論,修改,討論......)
    • 編碼解決(bug修復(fù)代碼和測(cè)試代碼)
    • 測(cè)試
    • 提交PR等待更多人review本次PR
  • 修復(fù)文檔錯(cuò)誤

    • 描述文檔錯(cuò)誤或文檔和實(shí)現(xiàn)不一致的內(nèi)容
    • 修改文檔
    • 提交PR等待更多人review本次PR
  • 處理review意見

    • 理解review意見
    • 和提出人確認(rèn)review意見及下一步修改方案
    • 重新提交代碼或文檔

PR 順利合并的關(guān)鍵是:溝通,溝通,溝通。

困難

個(gè)人認(rèn)為最大的困難是用英語溝通,尤其是口語化的英語更難準(zhǔn)確理解他人要表達(dá)的意思,Google翻譯已經(jīng)無法滿足訴求了。好一點(diǎn)的是,不管你的英語水平怎么樣,他們都能理解你要表達(dá)的意思。

向開源社區(qū)review學(xué)習(xí)什么?

  • 讓最懂的人review代碼
  • owner或committer提前參與方案的討論
  • 看護(hù)代碼架構(gòu),保持代碼架構(gòu)的一致性,摒棄“又不是不能用的”的思想。
  • 看護(hù)代碼的可擴(kuò)展性
  • 測(cè)試用例也要認(rèn)真review
  • 注釋也是review的重點(diǎn),保證注釋的準(zhǔn)確性至關(guān)重要。
最后編輯于
?著作權(quán)歸作者所有,轉(zhuǎn)載或內(nèi)容合作請(qǐng)聯(lián)系作者
【社區(qū)內(nèi)容提示】社區(qū)部分內(nèi)容疑似由AI輔助生成,瀏覽時(shí)請(qǐng)結(jié)合常識(shí)與多方信息審慎甄別。
平臺(tái)聲明:文章內(nèi)容(如有圖片或視頻亦包括在內(nèi))由作者上傳并發(fā)布,文章內(nèi)容僅代表作者本人觀點(diǎn),簡書系信息發(fā)布平臺(tái),僅提供信息存儲(chǔ)服務(wù)。

相關(guān)閱讀更多精彩內(nèi)容

友情鏈接更多精彩內(nèi)容