lvhongqiang / icp2_lhq

0 stars 0 forks source link

协作issue #1

Open lvhongqiang opened 6 years ago

maohuaming commented 6 years ago

@lvhongqiang ICP流程认证正式开始。开始时间:2018-04-03 13:50.

maohuaming commented 6 years ago

@lvhongqiang 请新建一个需求开发类的issue。

lvhongqiang commented 6 years ago

issue 的创建 需求类 issue: 由 committer 创建; 具体任务类 issue: 由 committer/contributor 创建; bug 报告类 issue: 可由任何人创建;

lvhongqiang commented 6 years ago

是不是得你来创建?

lvhongqiang commented 6 years ago

需求开发类的issue是不是得你来创建?@maohuaming

maohuaming commented 6 years ago

@lvhongqiang issue创建好了吗?

maohuaming commented 6 years ago

@lvhongqiang 注意一下:issue建好以后,要在comment@我一下。

maohuaming commented 6 years ago

@lvhongqiang 已打上ICP标签,请进行下一步。

maohuaming commented 6 years ago

@lvhongqiang 已打上size标签。请按照流程进行下一步。这是关键的一步,请注意。

lvhongqiang commented 6 years ago

已提交,准备pull request,有个问题,没有bug Issue,只写开发issue编号吗?

maohuaming commented 6 years ago

@lvhongqiang bug issue和需求issue两个是独立的。

maohuaming commented 6 years ago

@lvhongqiang 你上传的文件在哪里?你创建的分支在哪里?

maohuaming commented 6 years ago

@lvhongqiang default

maohuaming commented 6 years ago

@lvhongqiang 这段话,你没仔细研究吗?

lvhongqiang commented 6 years ago

README行吗?

maohuaming commented 6 years ago

@lvhongqiang default

maohuaming commented 6 years ago

@lvhongqiang 这段话,对上传的文件明确做了说明。你没看吗?

maohuaming commented 6 years ago

@lvhongqiang 你pull的分支建在哪里的?

lvhongqiang commented 6 years ago

develop

lvhongqiang commented 6 years ago

image

lvhongqiang commented 6 years ago

image

lvhongqiang commented 6 years ago

到底按哪个要求来?

maohuaming commented 6 years ago

@lvhongqiang 应该是:上传文件时,新建一个分支,分支名:feature/lvhongqiang-XXXX,把文件上传到这个分支,然后是提交pr,由feature/lvhongqiang-XXXX推送至develop。

maohuaming commented 6 years ago

@lvhongqiang 你不能只看流程图,要结合文档一起看。

lvhongqiang commented 6 years ago

逗我呢是么

georgeliuyu commented 6 years ago

@lvhongqiang 我们采用的是master和develop双分支方式管理。

georgeliuyu commented 6 years ago

也是希望通过这个测试让大家熟悉小白的开发流程。

lvhongqiang commented 6 years ago

我明白,熟悉流程很有必要,也正是我需要的,但我现在感觉到的并不是熟悉流程,而是在躲各种各样的坑。

lvhongqiang commented 6 years ago

一不留神就out了

georgeliuyu commented 6 years ago

@lvhongqiang @maohuaming 一会我们电话沟通一下

lvhongqiang commented 6 years ago

上周已经延期一次了,这次估计又要延期,一直拖着,实在对不住您@georgeliuyu

lvhongqiang commented 6 years ago

@maohuaming 抱歉,我脾气太急了,我已经提交了新的分支,咱们继续?

maohuaming commented 6 years ago

@lvhongqiang 继续,你提交pr吧。

lvhongqiang commented 6 years ago

pr已提交 @maohuaming

maohuaming commented 6 years ago

@lvhongqiang 已经处理。请进行merge操作。

lvhongqiang commented 6 years ago

@maohuaming 已merge

maohuaming commented 6 years ago

@lvhongqiang 注意一下:pull request 中包含的 commit 需正确命名,声明 #<issue 编号>。即创建pr的时候,commit的名称要以这个格式命名:for #<issue 编号>。在pr的comment中要引用一下issue,格式如下: default

lvhongqiang commented 6 years ago

晚了。。。我下次注意

maohuaming commented 6 years ago

认证暂停,暂停时间:2018-04-03 15:56。

lvhongqiang commented 6 years ago

又提交了一遍pr @maohuaming

maohuaming commented 6 years ago

认证继续,继续时间:2018-04-03 16:23

maohuaming commented 6 years ago

@lvhongqiang 请新建一个bug issue。

lvhongqiang commented 6 years ago

已创建bug issue @maohuaming

maohuaming commented 6 years ago

@lvhongqiang 已处理,请进行下一步操作。

lvhongqiang commented 6 years ago

已申请size @maohuaming

maohuaming commented 6 years ago

@lvhongqiang 请上传文件。

lvhongqiang commented 6 years ago

已上传文件 @maohuaming

maohuaming commented 6 years ago

@lvhongqiang 有几个问题: 1.pr引用的issue编号错了,应该是#5。 2.引用issue的格式没有按照这个格式来: default 3.pr的commit名称里没有声明issue编号。

maohuaming commented 6 years ago

@lvhongqiang 还有bug类型的issue,分支名称应该按照下面格式命名:bugfix/lvhongqiang-XXX

lvhongqiang commented 6 years ago

我还没提交pr呢 @maohuaming bugfix/lvhongqiang-layer-error 有问题吗?