You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/02/25 16:07:34 UTC

[GitHub] [trafficserver] brbzull0 opened a new pull request #8704: traffic_ctl - Rename --format parameter to --output.

brbzull0 opened a new pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704


   Makes more sense to call this `--output` as it's for setting the output format, this also performs some clean up on the  output style parameters making it easy to use.
   `all,req,resp` are now grouped into `rpc`.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] brbzull0 commented on pull request #8704: traffic_ctl - Rename --format parameter to --output.

Posted by GitBox <gi...@apache.org>.
brbzull0 commented on pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704#issuecomment-1055641700


   > We had a discussion in the weekly PR/issue review meeting and most people thought `--output` would mean sending it to a file. Leaving it as `--format` or `--output-format` might be better.
   
   fair enough, will leave it as `--format` then!
   Thanks!
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] brbzull0 merged pull request #8704: traffic_ctl - Use better format output options.

Posted by GitBox <gi...@apache.org>.
brbzull0 merged pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] bneradt commented on pull request #8704: traffic_ctl - Rename --format parameter to --output.

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704#issuecomment-1051238748


   [approve ci autest]


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] bryancall commented on pull request #8704: traffic_ctl - Rename --format parameter to --output.

Posted by GitBox <gi...@apache.org>.
bryancall commented on pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704#issuecomment-1055614870


   We had a discussion in the weekly PR/issue review meeting and most people thought `--output` would mean sending it to a file.  Leaving it as `--format` or `--output-format` might be better.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] bneradt removed a comment on pull request #8704: traffic_ctl - Rename --format parameter to --output.

Posted by GitBox <gi...@apache.org>.
bneradt removed a comment on pull request #8704:
URL: https://github.com/apache/trafficserver/pull/8704#issuecomment-1051238748


   [approve ci autest]


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org