New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

協作流程 #11

Open

fukuball opened this issue Jun 22, 2016 · 6 comments

Labels

討論

Collaborator

fukuball commented Jun 22, 2016

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

Collaborator Author

fukuball commented Jun 22, 2016

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

fukuball added the 討論 label

Collaborator Author

fukuball commented Jun 22, 2016

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

Collaborator

drumrick commented Jun 22, 2016

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

Collaborator Author

fukuball commented Jun 22, 2016

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

Collaborator

drumrick commented Jun 22, 2016

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

Collaborator Author

fukuball commented Jun 22, 2016

@drumrick 酷耶！

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment