Copilot commented on code in PR #4054:
URL: https://github.com/apache/solr/pull/4054#discussion_r4474018067


##########
dev-tools/scripts/smokeTestRelease.py:
##########
@@ -1041,6 +1100,8 @@ def parse_config():
                       help='Only perform download and sha hash check steps')
   parser.add_argument('--dev-mode', action='store_true', default=False,
                       help='Enable dev mode, will not check branch 
compatibility')
+  parser.add_argument('--skip-changelog', action='store_true', default=False,
+                      help='Skip checks that require the changelog to be 
prepared for this release (logchange.py prepare), e.g. when smoke testing a 
build made before the RM ran it')

Review Comment:
   The PR description specifies `--skip-changelog-current`, but this parser 
exposes only `--skip-changelog` (and the README/re-run hint follow that shorter 
name). A Jenkins invocation using the documented option will fail with an 
unrecognized-argument error, so the intended opt-out is not available under the 
published interface. Please use one name consistently across the parser, help 
text, README, and reuse hint.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to