Navigation Menu

Skip to content

Instantly share code, notes, and snippets.

@lookis
Last active February 21, 2021 10:13
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 1 You must be signed in to fork a gist
  • Save lookis/77bf0ee3ba762b1bfbd4 to your computer and use it in GitHub Desktop.
Save lookis/77bf0ee3ba762b1bfbd4 to your computer and use it in GitHub Desktop.
代码管理与规范

代码工作流程

1.先把要修改的工程fork到自己的github账号上.

2.在具体解决某个issue之前,先在自己的工程里创建一个分支,分支名字建议和issue的id相关以方便自己维护

3.在这个分支上具体解决该issue, 自己做好review之后提交一个pull-request

4.在pull-request被merge之前,如果需要解决一个新的issue,就再建立新的branch来工作,如果临时需要修改之前的那个提交,记得切换回刚才的branch

5.pull-request被review并merge了之后,大家就都更新一下自己的master代码,需要解决冲突的就解决冲突,没冲突的就继续开发。

即是: 一个issue对应一个本地branch对应多个commit

代码要求:

1.在提交的pull-request里要包含单元测试,没做单元测试的需要看代码的话直接私下分享link,因为如果你提交的pull-request被他人merge了,而又没有单元测试,很有可能别人的代码会破坏掉你的逻辑,所以pull-request到主版本库就是最后一次距离上线的提交了,一定要带单元测试

2.提交的pull-request里每个commit的comment都要精简地描述解决的问题和提交的内容

3.不要copy代码,因为我们很有可能有一天会把代码开源了,不想遇到各种license的法律问题,在提交的commit里该删的删,改的改,再提交

4.请符合.jshintrc规范,有改进的主意可以提出并讨论

@scxkd2016
Copy link

6

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