Skip to content

Commit 3782600

Browse files
authored
Merge pull request #451 from codacy/fix-logs-not-showing
fix(cli): logs early in the application start are now showing correctly
2 parents 1194767 + f27591c commit 3782600

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

cli/src/main/scala/com/codacy/analysis/cli/Main.scala

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package com.codacy.analysis.cli
22

33
import com.codacy.analysis.cli.analysis.ExitStatus
44
import com.codacy.analysis.cli.command.{AnalyseCommand, CLIApp, _}
5+
import com.codacy.analysis.core.utils.Logger
56

67
object Main extends MainImpl()
78

@@ -11,6 +12,8 @@ class MainImpl(analyseCommand: Analyze => AnalyseCommand = AnalyseCommand(_, sys
1112
extends CLIApp {
1213

1314
override def run(command: CommandOptions): ExitStatus.ExitCode = {
15+
Logger.setLevel(command.options.verboseValue)
16+
1417
runCommand(command)
1518
}
1619

cli/src/main/scala/com/codacy/analysis/cli/command/AnalyseCommand.scala

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ import com.codacy.analysis.core.git.{Commit, Git, Repository}
2727
import com.codacy.analysis.core.model._
2828
import com.codacy.analysis.core.storage.FileDataStorage
2929
import com.codacy.analysis.core.upload.ResultsUploader
30-
import com.codacy.analysis.core.utils.Logger
3130
import com.codacy.analysis.core.utils.SeqOps._
3231
import com.codacy.toolRepository.remote.ToolRepositoryRemote
3332
import com.codacy.toolRepository.remote.storage.ToolPatternsSpecsEncoders._
@@ -97,9 +96,6 @@ class AnalyseCommand(analyze: Analyze,
9796
configuration: CLIConfiguration,
9897
analyseExecutor: AnalyseExecutor,
9998
uploaderOpt: Either[String, Option[ResultsUploader]]) {
100-
101-
Logger.setLevel(analyze.options.verboseValue)
102-
10399
private val logger: org.log4s.Logger = getLogger
104100

105101
def run(): ExitStatus.ExitCode = {

0 commit comments

Comments
 (0)