提交初审注意事项

适用范围

本文适用于选拔赛任务,包含 Android、iOS、前端、PHP。框架任务不在此范围内,仅适用于完成框架任务后的具体项目任务。

1. 代码格式

  1. 趣编程所有项目,目前使用 2 空格缩进。
  2. 代码格式规范前端、PHP请认真阅读 PSR、Laravel 相关规范性文档,并按照规范编写。 其他方向,请参考技术小组博客中对应的规范编写。
  3. 代码格式可以参考现有代码。
  4. 在提交初审之前,请务必 逐行逐字 检查你的代码格式,养成良好的代码习惯。
  5. 趣编程目前的项目 api 使用 Repository 模式实现业务逻辑和数据访问的分离

2. 代码功能

  1. 任务问题,如果产品需求不懂,可以提问,请在 BC 对应项目群组(一般为 # 趣编程-讨论)或者 QQ 讨论组中提问。
  2. CR 不提供技术指导义务,一般会以关键点的方式提示,并要求你自己搜索。如果你提的问题有价值,相信 CR 会很乐意和你讨论。
  3. 如果你觉得交流困难,可以看看这个新人提问指北。本文本来是给趣编程学员提供如何提问的,可以看看。有问题,可以留言或者@林金晨。
  4. 提交初审前,烦请将功能测试成功的截图附在任务 card 中,并且不要作为card 封面图(移除封面图/Remove Cover)。
  5. 代码功能未实现提交初审的,直接打回重写

3. 其他常见问题

  1. 不要随意删除、修改其他人的或是与你功能无关的代码。
  2. 不要增加与功能、格式无关的空行和缩进。
  3. 不要作无意义的修改。

4. 提交规范

  1. 把 Trello 中的card 挪至相关的初审栏目中。
  2. card 上方请附上你的 PR 链接,务必保证链接是正确的,card 中没有写 PR 链接的直接打回重写
  3. PR 未 merge 之前,请每天请抽一点点时间查看一下 PR 是否有冲突,如有冲突请及时自行解决,不要等到初审的时候 CR 提醒冲突问题。 解决冲突办法,自己搜索技术小组博客.
  4. 每次提交审核前请务必测试代码功能无误,这是原则问题!
  5. 提交初审,请在 BC 中,项目的对应群组里 @CR,并附上你任务 card 的链接。

5. 其他相关

一般审核时间是1-2天,CR 有有自己的任务和事情,可能不能及时审核你的任务。请自己关注下进度,定期告知下 CR 请他帮你审核。如果多次 push 你的 CR 他仍然没有审核的,请到 「趣编程-讨论」这个组@林金晨,说明情况。

以上描述最终目的是节省大家的时间,提高工作效率,望大家能予以配合

感谢!

发表评论

电子邮件地址不会被公开。 必填项已用*标注