Skip to content

Handle GitLocalRepoNotFoundException to skip unavailable repos - #112

Merged
myleshenderson merged 2 commits into
masterfrom
myles/MW-10497-handle-missing-repo
Mar 4, 2026
Merged

Handle GitLocalRepoNotFoundException to skip unavailable repos#112
myleshenderson merged 2 commits into
masterfrom
myles/MW-10497-handle-missing-repo

Conversation

@myleshenderson

@myleshenderson myleshenderson commented Mar 4, 2026

Copy link
Copy Markdown

Summary

  • Catches GitLocalRepoNotFoundException in the repo sync loop to skip unavailable repos instead of failing the entire ingest job
  • Adds unit tests for the skip behavior, state handling, and ensuring other exceptions still propagate

Test plan

  • Unit test: skips repo when GitLocal() raises GitLocalRepoNotFoundException and continues to next repo
  • Unit test: state is not written for skipped repos
  • Unit test: generic GitLocalException still propagates (not silently caught)

Depends on: minwareco/minware-singer-utils#22
Jira: MW-10497

🤖 Generated with Claude Code

Catch GitLocalRepoNotFoundException in the repo processing loop to allow
the tap to gracefully skip unavailable repositories and continue processing
other repos in the org.

Changes:
- Import GitLocalRepoNotFoundException from minware-singer-utils
- Wrap repo processing (GitLocal init through state write) in try-except
- Log warning when repo not found and continue to next repo
- State is only written for successfully processed repos

This prevents single missing repos from crashing the entire org's ingest job.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@myleshenderson myleshenderson added the low-risk Low risk changes not needing review label Mar 4, 2026
@myleshenderson
myleshenderson requested a review from aaboyd March 4, 2026 15:46
…ndling

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@myleshenderson
myleshenderson merged commit 3c48b29 into master Mar 4, 2026
1 of 2 checks passed
@myleshenderson
myleshenderson deleted the myles/MW-10497-handle-missing-repo branch March 4, 2026 16:07
@aaboyd

aaboyd commented Mar 5, 2026

Copy link
Copy Markdown

looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-risk Low risk changes not needing review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants