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

Do not include NativeSinks.NativeAot.cs without FeaturePerfTracing #96811

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

SingleAccretion
Copy link
Contributor

@SingleAccretion SingleAccretion commented Jan 10, 2024

Doing so results in duplicate type/method definition errors.

Discovered downstream (dotnet/runtimelab#2482).

Doing so results in duplicate type/method definition errors.
@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Jan 10, 2024
@ghost
Copy link

ghost commented Jan 10, 2024

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Doing so results in duplicate type/method definition errors.

Discovered downstream.

Author: SingleAccretion
Assignees: -
Labels:

area-NativeAOT-coreclr

Milestone: -

@SingleAccretion
Copy link
Contributor Author

Failure is #94728.

Copy link
Member

@MichalStrehovsky MichalStrehovsky left a comment

Choose a reason for hiding this comment

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

Thank you!

@MichalStrehovsky
Copy link
Member

Failure is #94728.

My favorite!

@MichalStrehovsky MichalStrehovsky merged commit 8f79b66 into dotnet:main Jan 11, 2024
108 of 110 checks passed
tmds pushed a commit to tmds/runtime that referenced this pull request Jan 23, 2024
dotnet#96811)

Doing so results in duplicate type/method definition errors.
@SingleAccretion SingleAccretion deleted the No-FeaturePerfTracing branch January 30, 2024 12:52
@github-actions github-actions bot locked and limited conversation to collaborators Mar 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-NativeAOT-coreclr community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants