fix: raise GitConfigError on config read failures #162

Merged

fuzzy merged 1 commit from fix/126-gitconfigerror-unused into main

2026-07-06 08:23:54 +00:00

automation commented

2026-07-06 08:22:19 +00:00

Owner

Wraps get_user_name()/get_user_email() with try/except to raise GitConfigError instead of raw CalledProcessError. GitConfigError was previously defined in exceptions.py but never raised anywhere.

Closes #126

Wraps `get_user_name()`/`get_user_email()` with try/except to raise `GitConfigError` instead of raw `CalledProcessError`. `GitConfigError` was previously defined in `exceptions.py` but never raised anywhere. Closes #126

automation added 1 commit

2026-07-06 08:22:19 +00:00

fix(git): raise GitConfigError on config read failures

CI / lint-and-test (pull_request) Successful in 1m33s

Details

e35eb88384

Wrap get_user_name()/get_user_email() with try/except to
raise GitConfigError instead of raw CalledProcessError.
GitConfigError was previously defined but never raised.

Closes #126

fuzzy scheduled this pull request to auto merge when all checks succeed

2026-07-06 08:22:51 +00:00

the.auditor reviewed

2026-07-06 08:23:06 +00:00

the.auditor left a comment

Audit Review — PR #162

Clean fix for #126. get_user_name() and get_user_email() now catch CalledProcessError and raise GitConfigError with helpful config instructions. GitConfigError finally in use.

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #162 Clean fix for #126. `get_user_name()` and `get_user_email()` now catch `CalledProcessError` and raise `GitConfigError` with helpful config instructions. `GitConfigError` finally in use. No suggestions. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 08:23:09 +00:00

the.auditor left a comment

APPROVED. GitConfigError now in use, closes #126. Merge when ready.