SCANPY-190 Group command line arguments#218
Merged
maksim-grebeniuk-sonarsource merged 2 commits intomasterfrom Jun 16, 2025
Merged
SCANPY-190 Group command line arguments#218maksim-grebeniuk-sonarsource merged 2 commits intomasterfrom
maksim-grebeniuk-sonarsource merged 2 commits intomasterfrom
Conversation
maksim-grebeniuk-sonarsource
requested review from
ghislainpiot and
thomas-serre-sonarsource
| parser.add_argument( | ||
| "--sonar-links-homepage", "-Dsonar.links.homepage", type=str, help="The URL of the build project home page" | ||
| scanner_behavior_group.add_argument( | ||
| "-Dsonar.scm.forceReloadAll", |
Contributor
There was a problem hiding this comment.
This argument looks weird in the help documentation
-Dsonar.scm.forceReloadAll DSONAR.SCM.FORCERELOADALL
Equivalent to --sonar-scm-force-reload-all
this happens to all the ones with a getattr I think
Contributor
Author
There was a problem hiding this comment.
Thanks true and this could be an idea for another ticket
| "--sonar-scanner-java-exe-path", | ||
| "-Dsonar.scanner.javaExePath", | ||
|
|
||
| server_connection_group = parser.add_argument_group("SonarQube Server Connection") |
Contributor
There was a problem hiding this comment.
Suggested change
| server_connection_group = parser.add_argument_group("SonarQube Server Connection") | |
| server_connection_group = parser.add_argument_group("SonarQube Connection") |
| help="Python version used for the project", | ||
| ) | ||
| parser.add_argument( | ||
| reports_group = parser.add_argument_group("3rd party report arguments") |
Contributor
There was a problem hiding this comment.
Suggested change
| reports_group = parser.add_argument_group("3rd party report arguments") | |
| reports_group = parser.add_argument_group("3rd party reports arguments") |
I think it needs the s
joke1196
approved these changes
Jun 16, 2025
maksim-grebeniuk-sonarsource
force-pushed
the
SCANPY-190
branch
from
147391f to
d640a8d
Compare
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
SCANPY-190
Part of