-
-
Notifications
You must be signed in to change notification settings - Fork 6
feat: auto get base branch for create PR/MR #48
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.
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
Conversation
9aa32bf to
b10d5cb
Compare
|
应该是 close issue 吧 |
| .filter(function(branchName) { | ||
| return branchName !== cwb; | ||
| }); | ||
| return baseBranches.length >= 0 ? [baseBranches[0]] : []; |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
理论上来说,target branch 的来源分支应该可以有多个,最近的排第一个,但是还没完全搞清楚过滤这些分支的准确方法,所以预留了。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
那就加个 TODO 呗
bin/open-commander.js
Outdated
| // return name !== option.cwb; | ||
| // }); | ||
| } | ||
| if (remoteBranchLength > 1) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
那么 这个 >1 的逻辑能走到吗?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
现在走不到。预留的,之前想如果拿不到则拿所有的远程分支供选择。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO 吧
不然看起来很奇怪~
|
+1 |
b10d5cb to
68a2c37
Compare
|
2.7.0 |
feat: auto get base branch for create PR/MR
Close #47
This PR is create by
gitopen pr