mirror of
https://github.com/microsoft/semantic-kernel.git
synced 2026-10-12 02:19:06 +00:00
### Motivation and Context Makes the `logger` parameter optional in `TextCompletionClientBase.complete_async`, `TextCompletionClientBase.complete_stream_async`, `ChatCompletionClientBase.complete_chat_async`, and `ChatCompletionClientBase.complete_chat_stream_async` to match the current usage across the project where it's not passed. It also adds that `logger` optional parameter to the inheriting classes where needed to match the base class method signatures. Resolves #2509 ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [X] The code builds clean without any errors or warnings - [X] The PR follows the [SK Contribution Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [X] All unit tests pass, and I have added new tests where possible - [X] I didn't break anyone 😄