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

Fix build #2226

Merged
merged 2 commits into from
Aug 4, 2021
Merged

Fix build #2226

merged 2 commits into from
Aug 4, 2021

Conversation

reyang
Copy link
Member

@reyang reyang commented Aug 4, 2021

Part of #2222.

Changes

  • Updated the .csproj file and fixed all build warnings/errors under src.

@reyang reyang requested a review from a team August 4, 2021 06:36
@reyang reyang changed the base branch from main to aspnet-telemetrycorrelation August 4, 2021 06:36
@reyang reyang closed this Aug 4, 2021
@reyang reyang reopened this Aug 4, 2021
<!--PackageVersion></PackageVersion-->
<Authors>Microsoft</Authors>
<Title>Microsoft Asp.Net telemetry correlation</Title>
<TargetFrameworks>net461</TargetFrameworks>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cijothomas @rajkumar-rangaraj do you see a problem here if we target net461? Previously the target was <TargetFrameworks>net45</TargetFrameworks>.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be okay.
if anyone wants to use older than net461, then they can use the aspnet package.

Since Otel itself stopped support for older than net461, otel users shouldn't have issue either.

@reyang reyang merged commit 4b90945 into aspnet-telemetrycorrelation Aug 4, 2021
@reyang reyang deleted the reyang/build-pass branch August 4, 2021 16:35
cijothomas pushed a commit that referenced this pull request Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants