Skip to content

Commit

Permalink
CR
Browse files Browse the repository at this point in the history
  • Loading branch information
EyalDelarea committed Jan 1, 2025
1 parent bd32f2c commit 9e35c6c
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/main/java/io/jenkins/plugins/jfrog/JfStep.java
Original file line number Diff line number Diff line change
Expand Up @@ -103,8 +103,7 @@ protected String run() throws Exception {
EnvVars env = getContext().get(EnvVars.class);
Run<?, ?> run = getContext().get(Run.class);

// Initialize values to be used across the class
initClassValues(workspace, env, launcher);
initStaticVariables(workspace, env, launcher);

// Build the 'jf' command
ArgumentListBuilder builder = new ArgumentListBuilder();
Expand Down Expand Up @@ -133,13 +132,13 @@ protected String run() throws Exception {
}

/**
* Initializes values required across the class for running CLI commands.
* Initializes static variables there are required during runtime.
*
* @param workspace Workspace to use for any file operations.
* @param env Environment variables for this step.
* @param launcher Launcher to start processes.
*/
private void initClassValues(FilePath workspace, EnvVars env, Launcher launcher) throws IOException, InterruptedException {
private void initStaticVariables(FilePath workspace, EnvVars env, Launcher launcher) throws IOException, InterruptedException {
workspace.mkdirs();
isWindows = !launcher.isUnix();
jfrogBinaryPath = getJFrogCLIPath(env, isWindows);
Expand Down

0 comments on commit 9e35c6c

Please sign in to comment.