Thread (2 messages) flat view 2 messages, 2 authors, 2021-12-03

Re: [RFC PATCH 1/1] push: make '-u' have default arguments

From: Abhradeep Chakraborty <hidden>
Date: 2021-12-03 19:27:46

Junio C Hamano [off-list ref] wrote:
That of course is not an improvement but actively hurts them.  We
shouldn't be making it easier for our users to hurt themselves.
Hmm. In the scenario you mentioned, the proposed change is clearly
breaking. Thanks for notifying.
Shouldn't the rule be something like "if 'git push $args' (where
$args may be nothing, or any options other than '-u') pushes a
branch (or a set of branches) to a repository, 'git push -u $args'
(with the same $args) should set the branch.*.{remote,merge} for the
branch(es) to the same repository" for the introduction of default
to be truly an improvement?  Or is it too strict and makes the rule
not to trigger even for the intended audience?
Sounds good to me. But what if 'push.default' set to 'nothing'? Do the
proposed default arguments (I am saying about only the default arguments;
not the changes in code) are right fit for that case?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help