Skip to content

Conversation

@vsizov
Copy link
Contributor

@vsizov vsizov commented Jun 3, 2012

No description provided.

@vsizov
Copy link
Contributor Author

vsizov commented Jun 3, 2012

@randx Please do code review. Also need a little bit improve design of the checkbox.

@dzaporozhets
Copy link
Contributor

ok I'll do it in few days. Btw we should not allow delete of root_branch

@vsizov
Copy link
Contributor Author

vsizov commented Jun 4, 2012

Not sure that I rightly understood you. Do you mean default_branch?

@dzaporozhets
Copy link
Contributor

@vsizov Ex. If I do merge request from master(set as default branch) to stable i should not be able to remove master

@vsizov
Copy link
Contributor Author

vsizov commented Jun 4, 2012

i have added prevention to remove default_branch

@dzaporozhets
Copy link
Contributor

great. I'll take a look this evening

@dzaporozhets
Copy link
Contributor

screen

@ghost ghost assigned vsizov Jun 5, 2012
dzaporozhets added a commit that referenced this pull request Jun 5, 2012
ability to remove source branch after merge
@dzaporozhets dzaporozhets merged commit 73692f2 into master Jun 5, 2012
@vsizov
Copy link
Contributor Author

vsizov commented Jun 6, 2012

ÁÇÁ ËÒÕÔÏ

2012/6/5 Dmitriy Zaporozhets <
reply@reply.github.com

screen


Reply to this email directly or view it on GitHub:
#888 (comment)

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants