Skip to content

Commit 499476d

Browse files
committed
fix: delete workspace as needed
1 parent e3c188a commit 499476d

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

clone.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,6 @@ func (s *Server) prepareGitRepos(ctx context.Context, org, repo string, num int,
2727
return "", "", err
2828
}
2929

30-
defer func() {
31-
if s.debug { // debug mode, not delete workspace
32-
return
33-
}
34-
_ = os.RemoveAll(workspace)
35-
}()
36-
3730
refs, workDir := s.fixRefs(workspace, org, repo)
3831
log.Debugf("refs: %+v", refs)
3932
for _, ref := range refs {

server.go

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,6 +226,13 @@ func (s *Server) handleGitHubEvent(ctx context.Context, event *github.PullReques
226226
if err != nil {
227227
return err
228228
}
229+
defer func() {
230+
if s.debug { // debug mode, not delete workspace
231+
return
232+
}
233+
_ = os.RemoveAll(workspace)
234+
}()
235+
229236
info.workDir = workDir
230237
info.repoDir = workspace
231238

@@ -328,6 +335,12 @@ func (s *Server) handleGitLabEvent(ctx context.Context, event *gitlab.MergeEvent
328335
log.Errorf("prepare repo dir failed: %v", err)
329336
return ErrPrepareDir
330337
}
338+
defer func() {
339+
if s.debug { // debug mode, not delete workspace
340+
return
341+
}
342+
_ = os.RemoveAll(workspace)
343+
}()
331344
info.workDir = workDir
332345
info.repoDir = workspace
333346

0 commit comments

Comments
 (0)