-
Notifications
You must be signed in to change notification settings - Fork 287
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
Catching exception when failing to read config in partial trust #1119
Catching exception when failing to read config in partial trust #1119
Conversation
I'd love some insight as to why the build failed on this trivial change but the devops builds aren't publicly accessible. |
...icrosoft.ApplicationInsights/Extensibility/Implementation/Platform/PlatformImplementation.cs
Show resolved
Hide resolved
...icrosoft.ApplicationInsights/Extensibility/Implementation/Platform/PlatformImplementation.cs
Show resolved
Hide resolved
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.
azurepipelines infra is down. nothing to do with this PR. will merge after we get infra back.
@stimms The build is failing with stylecop checks. Can you build locally in Release, and you will be able to see the same errors. |
There is this test failure, about the new Warning trace. Test method Microsoft.ApplicationInsights.Extensibility.Implementation.Tracing.CoreEventSourceTest.MethodsAreImplementedConsistentlyWithTheirAttributes threw exception: Stack trace |
Failed MethodsAreImplementedConsistentlyWithTheirAttributes |
/AzurePipelines run |
For the Azure DevOps organization |
/azp help |
Supported commands help: Get descriptions, examples and documentation about supported commands Example: help "command_name" run: Run all pipelines or a specific pipeline for this repository using a comment. Use this command by itself to trigger all related pipelines, or specify a pipeline to run. Example: "run" or "run pipeline_name" See additional documentation. |
Fix Issue #416 .
Expand the area under try catch to allow for client initialization in partial trust scenarios, specifically as part of a Dynamics 365 plugin.