github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Remove unnecessary re-assignment of workDir variable

Browse Source
This commit is contained in:
Tasos Maschalidis
2025-10-17 19:14:08 +03:00
parent f40bcc37aa
commit ca03ea72f5
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
plugins/buildpacks/subcommands.go
View File

@@ -42,7 +42,6 @@ func CommandDetect(appName string, branch string) (err error) {
if err != nil { if err != nil {
return err return err
} }
workDir = checkedOutDir
defer func() { defer func() {
if err := os.RemoveAll(checkedOutDir); err != nil { if err := os.RemoveAll(checkedOutDir); err != nil {
common.LogWarn(fmt.Sprintf("Failed to remove temporary directory %s: %v", checkedOutDir, err)) common.LogWarn(fmt.Sprintf("Failed to remove temporary directory %s: %v", checkedOutDir, err))
@@ -51,7 +50,7 @@ func CommandDetect(appName string, branch string) (err error) {
dockerArgs := []string{ dockerArgs := []string{
"run", "--rm", "run", "--rm",
"-v", fmt.Sprintf("%s:/tmp/app", workDir), "-v", fmt.Sprintf("%s:/tmp/app", checkedOutDir),
"gliderlabs/herokuish", "/bin/herokuish", "buildpack", "detect", "/tmp/app", "gliderlabs/herokuish", "/bin/herokuish", "buildpack", "detect", "/tmp/app",
} }