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.

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

[#6825] Fix Microsoft.Bcl.AsyncInterfaces warnings #6837

Merged
merged 2 commits into from
Aug 5, 2024

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 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
10 checks passed
@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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: Jhonatan Sandoval Velasco <[email protected]>
Co-authored-by: Joel Mut <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

3 participants