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

Add Logging to CloudAdapter.ProcessAsync() - #6581

Closed
ram-xv wants to merge 1 commit into
mainfrom
ramfattah/CloudAdapterLogging
Closed

ram-xv wants to merge 1 commit into
mainfrom
ramfattah/CloudAdapterLogging

Conversation

@ram-xv

@ram-xv ram-xv commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

Fixes #6577

@ram-xv
ram-xv requested a review from a team as a code owner January 25, 2023 07:21
@ram-xv ram-xv added the Automation: No parity PR does not need to be applied to other languages. label Jan 25, 2023
@coveralls

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 338120

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 24 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.005%) to 73.353%

Files with Coverage Reduction New Missed Lines %
/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs 1 90.91%
/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs 1 90.91%
/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/ServiceCollectionExtensions.cs 1 93.55%
/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/CloudAdapter.cs 21 76.87%
Totals Coverage Status
Change from base Build 337253: -0.005%
Covered Lines: 23946
Relevant Lines: 32645

💛 - Coveralls

@BruceHaley

Copy link
Copy Markdown
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll

@ram-xv ram-xv closed this Feb 10, 2023
@ram-xv

ram-xv commented Feb 10, 2023

Copy link
Copy Markdown
Contributor Author

See #6587

@tracyboehrer
tracyboehrer (tracyboehrer) deleted the ramfattah/CloudAdapterLogging branch December 23, 2025 18:37
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.

Can you add a log line for this exception?

3 participants