Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Conversation

@sw-joelmut
Copy link
Collaborator

Addresses #6825
#minor

Description

This PR adds the Microsoft.Bcl.AsyncInterfaces package to Microsoft.Bot.Builder.Azure project, forcing 1.x version to 8.x, solving the MSB3277 warnings.

Specific Changes

  • Added Microsoft.Bcl.AsyncInterfaces to Microsoft.Bot.Builder.Azure project, forcing to use 8.x version instead of 1.x that caused the conflict.
  • Updated System.Diagnostics.DiagnosticSource in Microsoft.Bot.Builder.Integration.ApplicationInsights.WebApi project, solving MSB3245 warning.

Testing

The following images show the CI pipeline, and the reduction of the warnings.
image

@sw-joelmut sw-joelmut added the Automation: No parity PR does not need to be applied to other languages. label Aug 5, 2024
@sw-joelmut sw-joelmut requested a review from tracyboehrer August 5, 2024 14:02
@sw-joelmut sw-joelmut requested a review from a team as a code owner August 5, 2024 14:02
@BruceHaley
Copy link
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll

@tracyboehrer tracyboehrer merged commit 13bb5e1 into main Aug 5, 2024
@tracyboehrer tracyboehrer deleted the southworks/fix/async-interfaces-warnings branch August 5, 2024 14:26
tracyboehrer pushed a commit that referenced this pull request Sep 9, 2024
# Conflicts:
#	libraries/Microsoft.Bot.Builder.Azure/Microsoft.Bot.Builder.Azure.csproj
tracyboehrer added a commit that referenced this pull request Sep 10, 2024
* Fix for Skills endorsement check (#6846)

Co-authored-by: Tracy Boehrer <trboehre@microsoft.com>

* FederatedAppCredentials and factory

* add token credential authentication to BlobsTranscriptStore (#6813)

* [#6741] Remove .Net Core 3.1 from projects (#6819)

* Remove netcoreapp3.1 and fix System.Text.Json issue

* Fix System.Formats.Asn1 security issue

* Fix remaining System.Text.Json issue

* Removed unused argument from FederatedAuthenticator.CreateClientApplication

* Fix Microsoft.Bcl.AsyncInterfaces warning (#6837)

# Conflicts:
#	libraries/Microsoft.Bot.Builder.Azure/Microsoft.Bot.Builder.Azure.csproj

* add code ql comment about TypeNameHandling.None (#6843)

---------

Co-authored-by: Tracy Boehrer <trboehre@microsoft.com>
Co-authored-by: Jhonatan Sandoval Velasco <122501764+JhontSouth@users.noreply.github.com>
Co-authored-by: Joel Mut <62260472+sw-joelmut@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants