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
Webitools 198 add target compatible console version check before running the tools lfo #166
Webitools 198 add target compatible console version check before running the tools lfo #166
Conversation
…ble_Console_version_check_before_running_the_Tools_LFO' into WEBITOOLS-198_Add_target_compatible_Console_version_check_before_running_the_Tools_LFO # Conflicts: # aip-console-jenkins/src/test/java/io/jenkins/plugins/aipconsole/OnboardApplicationDeepAnalysisBuilderTest.java
…rsion_check_before_running_the_Tools_LFO
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.
Few review added
@@ -332,7 +339,7 @@ public AddVersionDescriptorImpl getDescriptor() { | |||
} | |||
|
|||
@Override | |||
public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { | |||
protected void performClient(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { |
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.
Is this class package private, since protected is being used?
@@ -172,7 +173,7 @@ public AnalyzeDescriptorImpl getDescriptor() { | |||
} | |||
|
|||
@Override | |||
public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { | |||
public void performClient(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { |
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.
in AddVersionBuilder it is protected and here it is public. Why this change in access modifier for same type of code?
@@ -83,20 +82,8 @@ public ArchitectureStudioBuilder(String applicationName) { | |||
} | |||
|
|||
@Override | |||
public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath filePath, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws | |||
public void performClient(@Nonnull Run<?, ?> run, @Nonnull FilePath filePath, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws |
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.
Same for this builder class
@Override | ||
public CreateAppDescriptorImpl getDescriptor() { | ||
return (CreateAppDescriptorImpl) super.getDescriptor(); | ||
} | ||
|
||
@Override | ||
public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { | ||
public void performClient(@Nonnull Run<?, ?> run, @Nonnull FilePath workspace, @Nonnull Launcher launcher, @Nonnull TaskListener listener) throws InterruptedException, IOException { |
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.
same issue
No description provided.