[GitHub] [cassandra] adelapena commented on a diff in pull request #2367: Generate.sh -s param to skip autodetection of tests

["adelapena (via GitHub)" <gi...@apache.org>]

adelapena commented on code in PR #2367:
URL: https://github.com/apache/cassandra/pull/2367#discussion_r1222911378


##########
.circleci/generate.sh:
##########
@@ -30,12 +30,13 @@ die ()
 
 print_help()
 {
-  echo "Usage: $0 [-f|-p|-a|-e|-i]"
+  echo "Usage: $0 [-f|-p|-a|-e|-i|-s]"
   echo "   -a Generate the config.yml, config.yml.FREE and config.yml.PAID expanded configuration"
   echo "      files from the main config_template.yml reusable configuration file."
   echo "      Use this for permanent changes in config that will be committed to the main repo."
   echo "   -f Generate config.yml for tests compatible with the CircleCI free tier resources"
   echo "   -p Generate config.yml for tests compatible with the CircleCI paid tier resources"
+  echo "   -s Skip auto injection of changed tests. Useful when you need to repeat a single one i.e."

Review Comment:
   Note that there is a second line in my suggestion above mentioning another use case for the new `-s`, which is when there are too many tests for CircleCI. That's what we have found in large patches like the ones for Tries and SAI.



-- 
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