Skip to content

Commit 88956bf

Browse files
committed
Split repo deletion into separate pipeline step #660
Signed-off-by: Jono Yang <[email protected]>
1 parent 289c336 commit 88956bf

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

minecode_pipelines/pipelines/mine_maven.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
# Visit https://github.com/aboutcode-org/scancode.io for support and download.
2222

2323

24+
from minecode_pipelines import pipes
2425
from minecode_pipelines.pipes import maven
2526
from scanpipe.pipelines.publish_to_federatedcode import PublishToFederatedCode
2627

@@ -41,8 +42,16 @@ class MineMaven(PublishToFederatedCode):
4142
def steps(cls):
4243
return (
4344
cls.check_federatedcode_eligibility,
45+
cls.clone_repos,
4446
cls.collect_packages_from_maven,
47+
cls.delete_cloned_repos,
4548
)
4649

50+
def clone_repos(self):
51+
self.repos = maven.clone_repos(self.project, self.log)
52+
4753
def collect_packages_from_maven(self):
4854
maven.collect_packages_from_maven(self.project, self.log)
55+
56+
def delete_cloned_repos(self):
57+
pipes.delete_cloned_repos(repos=self.repos, logger=self.log)

minecode_pipelines/pipes/maven.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -790,5 +790,5 @@ def collect_packages_from_maven(commits_per_push=10, logger=None):
790790
path=MAVEN_CHECKPOINT_PATH
791791
)
792792

793-
# clean up cloned repos
794-
pipes.delete_cloned_repos(repos=[data_repo, config_repo], logger=logger)
793+
repos_to_clean = [data_repo, config_repo]
794+
return repos_to_clean

0 commit comments

Comments
 (0)