search

ckmarkoh / deepSeg

A deep learning Chinese Word Segmentation toolkit
46 stars 18 forks source link

協作流程 #11

Open fukuball opened 8 years ago

fukuball commented 8 years ago
  1. 任一協作者認為有需要的工作事項，就可開 issue 出來，其他人有任何疑問可以在 issue 上討論，讓 issue 更明確
  2. 協作者可以自行認領 issue 進行工作，確認認領 issue 時請將 Assignees 設成認領者
  3. 為認領的 issue 開一個 branch 出來進行工作，branch 以 issue 進行命名
  4. Coding Style 使用 PEP8 標準，發出 pull request 前要用 pep8 套件做檢查再發出
  5. 工作完成發出 pull request 由其他任一協作者進行 review
  6. Review 時有任何問題可以及時反應，讓 pull request 可以及時修改
  7. 由 Reviewer 進行 merge，如果 merge 遇到 conflict 則通知發出 pull request 的人進行修復，之後再 merge
  8. Merge 完畢將 issue branch 刪除
fukuball commented 8 years ago

@ckmarkoh @drumrick 這是目前初步想到的協作流程，看看有沒有漏掉的地方，我們可以試行看看變做邊修～

fukuball commented 8 years ago

pep8 對於行的長度限制只有 80 字元，有點嚴格，行的長度限制這個部分我們要忽略嗎？

drumrick commented 8 years ago

pep8延伸是說可以到100 但是pycharm好像是用120 不管也是還好，現在螢幕很大 我都可以

fukuball commented 8 years ago

目前我是傾向忽略這條規則

drumrick commented 8 years ago

我寫了這個 pre-commit hook https://gist.github.com/drumrick/6c0d21817395c36143b453f88fe13406 只會檢查 cached *.py 檔案，忽略最大行數80的限制

fukuball commented 8 years ago

@drumrick 酷耶！