[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Update lilygit.tcl (Issue 2092) (issue 5504092)
From: |
graham |
Subject: |
Re: Update lilygit.tcl (Issue 2092) (issue 5504092) |
Date: |
Thu, 29 Dec 2011 06:45:53 +0000 |
Thanks for working on this! Unfortunately I'm between cities at the
moment; I'll try to comment more tomorrow evening.
First thought: I'm a bit leery of adding a "push to staging", since:
1. that clutters up the interface. Sure, it's just one more button, but
OTOH that's 25% more buttons. :)
2. it might confuse new contributors, who aren't allowed to push to
staging.
3. anybody who's stayed around long enough to get git push ability
shouldn't need it. (particularly once I get around to working on my cg
patch again)
http://codereview.appspot.com/5504092/
- Update lilygit.tcl (Issue 2092) (issue 5504092), Carl . D . Sorensen, 2011/12/29
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092),
graham <=
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), dak, 2011/12/29
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), pkx166h, 2011/12/29
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), Carl . D . Sorensen, 2011/12/29
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), Carl . D . Sorensen, 2011/12/29
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), graham, 2011/12/30
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), dak, 2011/12/30
- Re: Update lilygit.tcl (Issue 2092) (issue 5504092), Carl . D . Sorensen, 2011/12/30