Skip to content

Commit c529f07

Browse files
committed
Cleanup temporary files 🧹
1 parent e6698bb commit c529f07

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

‎cmd/gen-pipeline-report.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ var genPipelineReportCmd = &cobra.Command{
5757
logger.Error("Creating temporary directory failed %v", err)
5858
return err
5959
}
60+
defer os.RemoveAll(dir)
6061

6162
dir, err = cli.PullArtifacts("workflow", path.Join("test-results", pipelineID), dir, cmd)
6263
if err != nil {
@@ -82,13 +83,13 @@ var genPipelineReportCmd = &cobra.Command{
8283
return err
8384
}
8485

85-
defer os.deletefileName.
86-
8786
_, err = cli.PushArtifacts("workflow", fileName, path.Join("test-results", pipelineID+".json"), cmd)
8887
if err != nil {
8988
return err
9089
}
9190

91+
defer os.Remove(fileName)
92+
9293
return nil
9394
},
9495
}

‎cmd/publish.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,8 @@ var publishCmd = &cobra.Command{
146146
}
147147
}
148148

149+
defer os.Remove(fileName)
150+
149151
return nil
150152
},
151153
}

‎pkg/cli/cli.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ func MergeFiles(path string, cmd *cobra.Command) (*parser.Result, error) {
324324
}
325325

326326
// Load ...
327-
// [TODO]: TEST THIS!!!
327+
// [TODO]: Test this
328328
func Load(path string) (*parser.Result, error) {
329329
var result parser.Result
330330
jsonFile, err := os.Open(path)

0 commit comments

Comments
 (0)