Skip to content

Conversation

@hotoo
Copy link
Owner

@hotoo hotoo commented Feb 22, 2017

Close #47

This PR is create by gitopen pr

@hotoo hotoo force-pushed the feat/mr-basebranch branch from 9aa32bf to b10d5cb Compare February 22, 2017 10:03
@nightink
Copy link

应该是 close issue 吧

.filter(function(branchName) {
return branchName !== cwb;
});
return baseBranches.length >= 0 ? [baseBranches[0]] : [];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个地方的 返回的 array length 0 or 1?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

理论上来说,target branch 的来源分支应该可以有多个,最近的排第一个,但是还没完全搞清楚过滤这些分支的准确方法,所以预留了。

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那就加个 TODO 呗

// return name !== option.cwb;
// });
}
if (remoteBranchLength > 1) {
Copy link

@nightink nightink Feb 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那么 这个 >1 的逻辑能走到吗?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

现在走不到。预留的,之前想如果拿不到则拿所有的远程分支供选择。

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO 吧

不然看起来很奇怪~

@coveralls
Copy link

coveralls commented Feb 22, 2017

Coverage Status

Coverage decreased (-9.3%) to 83.333% when pulling b10d5cb on feat/mr-basebranch into 396e58b on master.

@coveralls
Copy link

coveralls commented Feb 22, 2017

Coverage Status

Coverage decreased (-9.3%) to 83.333% when pulling b10d5cb on feat/mr-basebranch into 396e58b on master.

@nightink
Copy link

+1

@hotoo hotoo force-pushed the feat/mr-basebranch branch from b10d5cb to 68a2c37 Compare February 23, 2017 02:48
@hotoo hotoo merged commit 31b2f5a into master Feb 23, 2017
@hotoo hotoo deleted the feat/mr-basebranch branch February 23, 2017 02:49
@hotoo
Copy link
Owner Author

hotoo commented Feb 23, 2017

2.7.0

hotoo added a commit that referenced this pull request Mar 3, 2017
feat: auto get base branch for create PR/MR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants