-
Notifications
You must be signed in to change notification settings - Fork 71
fix: add logging for start_session details #299
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.
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
fix: add logging for start_session details #299
Conversation
Signed-off-by: Paul S. Schweigert <[email protected]>
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
|
Looks good, except for the failing Please run the pre-commit hooks and push Thanks for the contribution! |
Signed-off-by: Paul S. Schweigert <[email protected]>
done! |
|
Thanks, looks good. We're in code freeze until #310 is merged. That should resolve automatically for this PR. I will merge this some time tomorrow, or ping you if there are unanticipated merge issues. |
|
#310 is now merged. Updated this branch and re-ran workflows. Will merge on success. Thanks for the patience. |
Fixes #39
Adds an info-level log message when start_session() is called, displaying the backend name, model ID (resolved for the specific backend), context type, and model options (if provided).