thoughtworks / metrik

An easy-to-use, cross-platform measurement tool that pulls data out of CD pipelines and analysis the four key metrics for you.
MIT License
354 stars 87 forks source link

fix: Synchronize failed while fetching commits of deleted branch in Github #122

Closed ATPEEE closed 2 years ago

ATPEEE commented 2 years ago

Our team fix sync failed error when fetching commits of deleted branch as the following ideas: #114

Also, we refactor GithubPipelineService following the picture below: WeChatWorkScreenshot_b00388a9-c06f-413d-a35f-1a7afff5b586

We extract several services from GithubPipelineService to make it independent. Thus, we add some unit tests according to new services and fix GithubPipelineServiceTest.