Skip to content

docs(codereview): add logging level rules#158

Merged
sohil-kshirsagar merged 1 commit intomainfrom
sohil/docs/bugbot-logging-level-rules
Apr 21, 2026
Merged

docs(codereview): add logging level rules#158
sohil-kshirsagar merged 1 commit intomainfrom
sohil/docs/bugbot-logging-level-rules

Conversation

@sohil-kshirsagar
Copy link
Copy Markdown
Contributor

@sohil-kshirsagar sohil-kshirsagar commented Apr 21, 2026

Mirror the logging-level guardrail added to the python SDK's code review guidelines (drift-python-sdk#95) so the same class of bug can't ship here either.

Mirror the logging-level guardrail added to the python SDK's BUGBOT
(Use-Tusk/drift-python-sdk#95). The python SDK had shipped per-connect
and per-request INFO logs that flooded customer output; this SDK's
instrumentations are currently clean, but the default log level is
"info" and the risk is the same. Adding the rules now so future
instrumentations don't regress.
@sohil-kshirsagar sohil-kshirsagar marked this pull request as ready for review April 21, 2026 20:59
@sohil-kshirsagar sohil-kshirsagar changed the title docs(bugbot): add logging level rules docs(codereview): add logging level rules Apr 21, 2026
@tusk-dev
Copy link
Copy Markdown
Contributor

tusk-dev Bot commented Apr 21, 2026

Code Review

Tusk Review: No issues found


Unit Tests

No tests generated

View output

No symbols were selected for unit test generation because the changes are limited to updates in .cursor/BUGBOT.md, a documentation file containing developer guidelines. There are no code or logic modifications, so there is nothing testable or requiring unit tests.

View check history

Commit Unit Tests Created (UTC)
7b213ad No tests generated · Output Apr 21, 2026 9:00PM

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@sohil-kshirsagar sohil-kshirsagar merged commit 09222c9 into main Apr 21, 2026
6 checks passed
@sohil-kshirsagar sohil-kshirsagar deleted the sohil/docs/bugbot-logging-level-rules branch April 21, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants