[GitHub] [cassandra] josh-mckenzie commented on pull request #1579: Cassandra 17563

[GitBox <gi...@apache.org>]

josh-mckenzie commented on PR #1579:
URL: https://github.com/apache/cassandra/pull/1579#issuecomment-1111198074

   Could we get more detail on the help output for these scripts? It's not clear what they're supposed to do, how they work, how to test, etc w/out opening them up. Nothing in readme.md and the help output is:
   ```./circle-ci-config.py
   usage: circle-ci-config.py [-h] [--stdout] target
   circle-ci-config.py: error: the following arguments are required: target
   ```
   
   Also, having "HIGHER" as a target when we have LOWRES, MIDRES, HIGHRES seems pretty confusing for me:
   ```
   51 targets = {
   52         'MIDRES': {
   ...
   89         'HIGHER': {
   ```


-- 
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: pr-unsubscribe@cassandra.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org