[Raphael Fakhri (RaphaelFakhri)]

Summary

When git remote show origin fails, _get_base_branch logs a warning with _logger.warning(f"...", e). The message has no %s specifier for the extra argument, so the logging module raises TypeError: not all arguments converted during string formatting and prints a --- Logging error --- traceback to stderr. This change formats the exception into the message.

Changes

• Put the exception in the f-string of the warning in _get_base_branch.
• Add a test that runs the failing lookup and checks that the warning formats and that the function falls back to main.

Testing

cd py
python -m pytest src/braintrust/test_gitutil.py

Before the change: 1 failed, 5 passed. After the change: 6 passed.

[Abhijeet Prasad (AbhiPrasad)]

released with https://github.com/braintrustdata/braintrust-sdk-python/releases/tag/py-sdk-v0.44.0