Skip to content
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

Jump to bottom

.Net: Fix a few issues with ChatHistoryExtensions #3508

Merged
merged 1 commit into from
Nov 15, 2023
Merged

.Net: Fix a few issues with ChatHistoryExtensions #3508

merged 1 commit into from
Nov 15, 2023

Commits on Nov 15, 2023

  1. Fix a few issues with ChatHistoryExtensions 

    1. It's public but has no public members. It shouldn't be public.
2. It allocates a bunch but doesn't actually need to allocate anything (at least when not passed a TokenCounter, which it currently never is).
    @stephentoub
    stephentoub committed Nov 15, 2023
    Configuration menu
    Copy the full SHA
    f23b8b3 View commit details
    Browse the repository at this point in the history