Skip to content

Commit 7c209a5

Browse files
review improvements
1 parent 3735b88 commit 7c209a5

File tree

4 files changed

+13
-6
lines changed

4 files changed

+13
-6
lines changed

config/config.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,10 @@ func (c *ConfigType) LocalCodacyDirectory() string {
4343
return c.localCodacyDirectory
4444
}
4545

46+
func (c *ConfigType) ToolsConfigsDirectory() string {
47+
return c.toolsConfigDirectory
48+
}
49+
4650
func (c *ConfigType) ProjectConfigFile() string {
4751
return c.projectConfigFile
4852
}

tools/eslintRunner.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package tools
22

33
import (
4+
"codacy/cli-v2/config"
45
"os"
56
"os/exec"
67
"path/filepath"
@@ -20,7 +21,7 @@ func RunEslint(repositoryToAnalyseDirectory string, eslintInstallationDirectory
2021
cmd.Env = append(cmd.Env, "ESLINT_USE_FLAT_CONFIG=true")
2122

2223
// Add config file from tools-configs directory
23-
configFile := filepath.Join(".codacy", "tools-configs", "eslint.config.mjs")
24+
configFile := filepath.Join(config.Config.ToolsConfigDirectory(), "eslint.config.mjs")
2425
cmd.Args = append(cmd.Args, "-c", configFile)
2526

2627
if autoFix {
@@ -48,9 +49,9 @@ func RunEslint(repositoryToAnalyseDirectory string, eslintInstallationDirectory
4849
nodePathEnv := "NODE_PATH=" + eslintInstallationNodeModules
4950
cmd.Env = append(cmd.Env, nodePathEnv)
5051

51-
//DEBUG
52-
//fmt.Println(cmd.Env)
53-
//fmt.Println(cmd)
52+
// DEBUG
53+
// fmt.Println(cmd.Env)
54+
// fmt.Println(cmd)
5455

5556
// TODO eslint returns 1 when it finds errors, so we're not propagating it
5657
cmd.Run()

tools/pmdRunner.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package tools
22

33
import (
4+
"codacy/cli-v2/config"
45
"os"
56
"os/exec"
67
"path/filepath"
@@ -24,7 +25,7 @@ func RunPmd(repositoryToAnalyseDirectory string, pmdBinary string, pathsToCheck
2425

2526
// Add config file from tools-configs directory if not specified
2627
if rulesetFile == "" {
27-
configFile := filepath.Join(".codacy", "tools-configs", "pmd-ruleset.xml")
28+
configFile := filepath.Join(config.Config.ToolsConfigDirectory(), "pmd-ruleset.xml")
2829
cmd.Args = append(cmd.Args, "-R", configFile)
2930
} else {
3031
cmd.Args = append(cmd.Args, "-R", rulesetFile)

tools/trivyRunner.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package tools
22

33
import (
4+
"codacy/cli-v2/config"
45
"os"
56
"os/exec"
67
"path/filepath"
@@ -11,7 +12,7 @@ func RunTrivy(repositoryToAnalyseDirectory string, trivyBinary string, pathsToCh
1112
cmd := exec.Command(trivyBinary, "fs")
1213

1314
// Add config file from tools-configs directory
14-
configFile := filepath.Join(".codacy", "tools-configs", "trivy.yaml")
15+
configFile := filepath.Join(config.Config.ToolsConfigDirectory(), "trivy.yaml")
1516
cmd.Args = append(cmd.Args, "--config", configFile)
1617

1718
// Add format options

0 commit comments

Comments
 (0)