You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@otava.apache.org by "Sanjays2402 (via GitHub)" <gi...@apache.org> on 2026/07/25 23:59:36 UTC

[PR] fix: move --update- args into their service option groups [otava]

Sanjays2402 opened a new pull request, #165:
URL: https://github.com/apache/otava/pull/165

   Closes #120
   
   `--update-grafana`, `--update-postgres` and `--update-bigquery` are added directly to the `analyze` subparser, so argparse lists them under the generic `options:` section instead of next to the service options they configure.
   
   The subparser already inherits the Grafana/PostgreSQL/BigQuery groups from the shared parent parser, so this adds a small `config.argument_group()` helper that looks up an existing group by title and registers each flag on the matching group. Parsing behaviour is unchanged — only the help layout differs. The golden help assertions in `tests/cli_help_test.py` are updated accordingly and fail without the fix.
   


-- 
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: commits-unsubscribe@otava.apache.org

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


Re: [PR] fix: move --update- args into their service option groups [otava]

Posted by "Gerrrr (via GitHub)" <gi...@apache.org>.
Gerrrr merged PR #165:
URL: https://github.com/apache/otava/pull/165


-- 
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: commits-unsubscribe@otava.apache.org

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