followup and wontfix this
This commit is contained in:
		
					parent
					
						
							
								96c269e087
							
						
					
				
			
			
				commit
				
					
						e08f170b0e
					
				
			
		
					 2 changed files with 24 additions and 1 deletions
				
			
		| 
						 | 
				
			
			@ -73,4 +73,4 @@ fsck: 1 failed
 | 
			
		|||
Yes, great tool ! Thanks !
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
> [[wontfix|done]] per my comment --[[Joey]]
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -0,0 +1,23 @@
 | 
			
		|||
[[!comment format=mdwn
 | 
			
		||||
 username="joey"
 | 
			
		||||
 subject="""comment 1"""
 | 
			
		||||
 date="2023-11-21T20:03:45Z"
 | 
			
		||||
 content="""
 | 
			
		||||
Hmm, I've never considered combining --quiet with --json. It's kind of
 | 
			
		||||
undefined and really not clear to me what it should do.
 | 
			
		||||
 | 
			
		||||
But, --json-error-messages makes the json contain an error-messages field,
 | 
			
		||||
and the error message is in there. So you can just extract that and ignore
 | 
			
		||||
the other messages in the json output. No need to use --quiet then.
 | 
			
		||||
 | 
			
		||||
I suppose there may be someone who uses --json as a matter of course, but
 | 
			
		||||
adds --quiet to that when they want to disable the json output. So
 | 
			
		||||
changing the current behavior, ill-defined as it is, would be asking for
 | 
			
		||||
trouble.
 | 
			
		||||
 | 
			
		||||
What actually happens currently is which ever output option comes last
 | 
			
		||||
overrides earlier options. So `--json --quiet` is quiet, and `--quiet
 | 
			
		||||
--json` outputs json. `--json-error-messages` is like `--json` in this
 | 
			
		||||
regard to. Which is just behavior that fell out of the option parser
 | 
			
		||||
implementation.
 | 
			
		||||
"""]]
 | 
			
		||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue