Skip to content

add AddCallerSkip(1) so we aren't having the logger log itself #2182

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
matthewlouisbrockman wants to merge 3 commits into
base: main
Choose a base branch
from

need nolint on the tests

b9136c6
Select commit
Loading
Failed to load commit list.
Draft

add AddCallerSkip(1) so we aren't having the logger log itself #2182

need nolint on the tests
b9136c6
Select commit
Loading
Failed to load commit list.
Cursor / Cursor Bugbot completed Mar 20, 2026 in 7m 10s

Bugbot Review

Bugbot Analysis Progress (7m 12s elapsed)

✅ Gathered PR context (2s)
✅ Completed bug detection (7m 4s)
✅ Posted analysis results (6s)

Final Result: Bugbot completed review and found 1 potential issue.

Request ID: serverGenReqId_53aceeaf-f387-4586-b49f-8a63e21e01d8

Details

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

View more details on Cursor