SCANPY-214: Change property name --sonar-cpd-python-minimum-tokens to --sonar-cpd-py-minimum-tokens

[joke1196]

SCANPY-214

Part of

[Seppli11]

The change itself looks good. However, this would be a breaking change.
As such, I think it would make sense to keep both --sonar-cpd-py-minimum-lines and --sonar-cpd-python-minimum-lines (the same applies for the minimum tokens).

This would still be a breaking change for the environment variable parsing, but I feel like we could live with that

[Seppli11]

LGTM! thanks for the quick fix.

I think we don't need the -D... args, since they just act as documentation.

(Sorry, I probably could have been a bit clearer in my review)

[Seppli11]

The -Dsonar.cpd.python.minimumLines we shouldn't need, since any non-consumed parameter beginning with -D... will be passed along.

Same applies for minimum tokens.

[joke1196]

I think we should keep them as we will pass the property to the correct sonar.cpd.py....

[Seppli11]

True. I think the PR can be merged then

[sonarqube-next]

Quality Gate passed for 'Python Scanner'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube