-
Notifications
You must be signed in to change notification settings - Fork 285
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
add support for Net462 #2277
add support for Net462 #2277
Conversation
BASE/src/ServerTelemetryChannel/Implementation/ApplicationFolderProvider.cs
Outdated
Show resolved
Hide resolved
...icationInsights/Extensibility/Implementation/Tracing/BaseDefaultHeartbeatPropertyProvider.cs
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
BASE/src/Microsoft.ApplicationInsights/Microsoft.ApplicationInsights.csproj
Show resolved
Hide resolved
@@ -2,7 +2,7 @@ | |||
<Import Project="$(PropsRoot)\Product.props" /> | |||
|
|||
<PropertyGroup> | |||
<TargetFrameworks>net452</TargetFrameworks> | |||
<TargetFrameworks>net452;net462</TargetFrameworks> |
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.
Adding net462
might force us to support this project Microsoft.ApplicationInsights.EtwCollector
for a long time. Should we reconsider adding net462 to this project?
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.
I agree with you, but I was reluctant to make that decision.
Future Proof related to #2273.
Helping to support AAD work #2190.
Changes
Checklist
For significant contributions please make sure you have completed the following items:
The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:
Notes for reviewers:
/AzurePipelines run
will queue all builds/AzurePipelines run <pipeline-name>
will queue a specific build