threads / patch / 60409

patchgit-push: more visibility for -q option

Subject: [PATCH] git-push: more visibility for -q option

## tl;dr

4 messages between Oct 20, 2023 and Oct 30, 2023. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Michal Suchanek· Oct 20, 2023, 18:45 UTC · lore
The -v option listed at the top as option al parameter while -q is not.
List -q alongside -v.
Signed-off-by: Michal Suchanek <msuchanek@suse.de>
---
 Documentation/git-push.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to Documentation/git-push.txt +1 −1
diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt
index 5b4edaf4a8..003bc7d9ce 100644
--- a/Documentation/git-push.txt
+++ b/Documentation/git-push.txt
@@ -10,7 +10,7 @@ SYNOPSIS
 --------
 [verse]
 'git push' [--all | --branches | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=<git-receive-pack>]
-	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]
+	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-q | --quiet] [-v | --verbose]
 	   [-u | --set-upstream] [-o <string> | --push-option=<string>]
 	   [--[no-]signed|--signed=(true|false|if-asked)]
 	   [--force-with-lease[=<refname>[:<expect>]] [--force-if-includes]]
-- 
2.42.0
Eric Sunshine· Oct 20, 2023, 20:05 UTC · re: Michal Suchanek · lore

Re: [PATCH] git-push: more visibility for -q option

On Fri, Oct 20, 2023 at 2:46 PM Michal Suchanek <msuchanek@suse.de> wrote:
> The -v option listed at the top as option al parameter while -q is not.
s/option al/optional/
> List -q alongside -v.
>
> Signed-off-by: Michal Suchanek <msuchanek@suse.de>
Junio C Hamano· Oct 20, 2023, 20:08 UTC · re: Michal Suchanek · lore

Re: [PATCH] git-push: more visibility for -q option

Michal Suchanek <msuchanek@suse.de> writes:
> The -v option listed at the top as option al parameter while -q is not.
"as option al parameter" - ECANNOTPARSE.  Probably
    The `-v` option is shown in the SYNOPSIS section near the top,
    but `-q` is not shown anywhere there.

or something, I think. I agree showing it next to "-v" would make the most sense.

Show 18 quoted lines
>
> List -q alongside -v.
>
> Signed-off-by: Michal Suchanek <msuchanek@suse.de>
> ---
>  Documentation/git-push.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt
> index 5b4edaf4a8..003bc7d9ce 100644
> --- a/Documentation/git-push.txt
> +++ b/Documentation/git-push.txt
> @@ -10,7 +10,7 @@ SYNOPSIS
>  --------
>  [verse]
>  'git push' [--all | --branches | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=<git-receive-pack>]
> -	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]
> +	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-q | --quiet] [-v | --verbose]

Yup, the change makes sense. We may want to wrap the first line to a more reasonable length in a separate commit, and when that happens, we probably would want to start [-v] [-q] on a separate line as well, but for now this would do.

Michal Suchánek· Oct 30, 2023, 10:40 UTC · re: Junio C Hamano · lore

Re: [PATCH] git-push: more visibility for -q option

Hello,
On Fri, Oct 20, 2023 at 01:08:04PM -0700, Junio C Hamano wrote:
Show 11 quoted lines
> Michal Suchanek <msuchanek@suse.de> writes:
> 
> > The -v option listed at the top as option al parameter while -q is not.
> 
> "as option al parameter" - ECANNOTPARSE.  Probably
> 
>     The `-v` option is shown in the SYNOPSIS section near the top,
>     but `-q` is not shown anywhere there.
> 
> or something, I think.  I agree showing it next to "-v" would make
> the most sense.

when rebasing to current master I see this is already applied with the fixed commit message, great!

Thanks
Michal
Show 25 quoted lines
> 
> >
> > List -q alongside -v.
> >
> > Signed-off-by: Michal Suchanek <msuchanek@suse.de>
> > ---
> >  Documentation/git-push.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt
> > index 5b4edaf4a8..003bc7d9ce 100644
> > --- a/Documentation/git-push.txt
> > +++ b/Documentation/git-push.txt
> > @@ -10,7 +10,7 @@ SYNOPSIS
> >  --------
> >  [verse]
> >  'git push' [--all | --branches | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=<git-receive-pack>]
> > -	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]
> > +	   [--repo=<repository>] [-f | --force] [-d | --delete] [--prune] [-q | --quiet] [-v | --verbose]
> 
> Yup, the change makes sense.  We may want to wrap the first line to
> a more reasonable length in a separate commit, and when that
> happens, we probably would want to start [-v] [-q] on a separate
> line as well, but for now this would do.
> 

← back to recent threads