New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ITs: Deduplicate orchestrator initialization #7017
Conversation
94ffff5
to
e948780
Compare
// The SonarQube alias "LTS" has been dropped. An alternative is "LATEST_RELEASE[6.7]". | ||
// The term "latest" refers to the highest version number, not the most recently published version. | ||
public static String replaceLtsVersion(String version) { | ||
if (version != null && version.equals("LTS")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use the old "LTS" value anywhere. And we don't need to => we don't need this functionality.
e948780
to
b88c6f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - only a comment regarding a a comment in the code that is not clear to me.
Kudos, SonarCloud Quality Gate passed! |
Kudos, SonarCloud Quality Gate passed! |
Deduplicate version configuration and orchestrator scaffolding to a single place.