- Notifications
You must be signed in to change notification settings - Fork845
Merge internal changes#6921
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Getting the branch ready for 9.10 release----#### AI description (iteration 1)#### PR ClassificationThis pull request prepares the branch for the 9.10 release by updating dependency versions and adjusting build configurations.#### PR SummaryThe update bumps multiple dependency versions from 9.0.9 to 9.0.10, updates release stabilization properties, and refines build pipeline tasks and NuGet configurations.- **`eng/Version.Details.xml` and `eng/Versions.props`**: Dependency versions and SHA values have been updated to 9.0.10, with build properties modified to enable release stabilization and update LTS version numbers.- **`azure-pipelines.yml`**: The code coverage stage has been removed and stage dependencies adjusted.- **`NuGet.config`**: Package source mappings were removed, and internal feed sources have been appropriately disabled.- **`eng/pipelines/templates/BuildAndTest.yml` & `Directory.Build.props`**: New tasks were added to set up private feed credentials, integration test steps were commented out due to authentication requirements, and NU1507 warnings have been suppressed.<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
* Initial plan* Implement local function name cleanup for AIFunctionFactoryCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Fix lambda name uniqueness by capturing full ordinal suffixCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Fix analyzer errors by adding comments to empty test methodsCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Change Assert.Contains to Assert.StartsWith for lambda name testsCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Consolidate regexes and improve test parameter namesCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Keep ordinal suffix for local functions to ensure uniquenessCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Fix build errors for netstandard2.0 and net462 targetsCo-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>* Cleanup---------Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>Co-authored-by: Stephen Toub <stoub@microsoft.com>
We're currently filtering down to only AITools that are AIFunctionDeclaration. With this, we'll handle AIFunctionDeclaration wrappers the same as well, and for all other tools emit their name as the type. This can be further extended as otel adds more details about how non-function tools should be represented.
* Update ToChatResponse{Async} to also factor in AuthorNameOriginally we were only factoring in Role. Then we also augmented it to factor in MessageId, since updates from different messages are by definition not part of the same message. Now it also makes sense to factor in AuthorName, as different text from different speakers / agents are also by definition not part of the same message.* Fix useAsync inversion in tests* add support for background responsesAdds a model for supporting background responses (long-running operations), updates the chat completion model to use it, and integrates this functionality into OpenAIResponsesChatClient.---------Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>Co-authored-by: Stephen Toub <stoub@microsoft.com>Co-authored-by: westey <164392973+westey-m@users.noreply.github.com>
All clean cherry-picks
…6915)It should fail gracefully by ignoring any content it can't output telemetry for rather than throwing.
Clean cherry-pick
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR merges internal changes that update dependency versions across the .NET Extensions repository. The primary purpose is to update Microsoft .NET runtime, ASP.NET Core, and Entity Framework Core packages from version 9.0.9 to 9.0.10, and LTS versions from 8.0.20 to 8.0.21.
Key changes include:
- Updated all .NET 9.0 runtime and extensions packages from 9.0.9 to 9.0.10
- Updated LTS (8.0) packages from 8.0.20 to 8.0.21
- Updated corresponding SHA references for the new package versions
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/Versions.props | Updated version properties for .NET runtime, ASP.NET Core, and Entity Framework packages |
| eng/Version.Details.xml | Updated dependency version details and corresponding SHA commit references |
4e17769 intodotnet:mainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Merging changes from internal
This should be merged and not squashed.
Microsoft Reviewers:Open in CodeFlow