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

port system.diagnostics.performanceCounter #55134

Merged
merged 2 commits into from
Jul 6, 2021
Merged

port system.diagnostics.performanceCounter #55134

merged 2 commits into from
Jul 6, 2021

Conversation

Anipik
Copy link
Contributor

@Anipik Anipik commented Jul 4, 2021

  • set privateAssets = all to avoid adding package dependencies on ns2.0
  • add _GetPackageFiles to before targets for NETStandardCompatError

Another example pr for reference.

https://github.com/Anipik/PackagePortTester

i pushed the package validator which could be run on package paths to test if the port is successful.

@ghost
Copy link

ghost commented Jul 4, 2021

Tagging subscribers to this area: @tarekgh, @tommcdon, @pjanotti
See info in area-owners.md if you want to be subscribed.

Issue Details
  • set privateAssets = all to avoid adding package dependencies on ns2.0
  • add _GetPackageFiles to before targets for NETStandardCompatError

Another example pr for reference.

Author: Anipik
Assignees: -
Labels:

area-System.Diagnostics.Tracing

Milestone: -

Directory.Build.targets Outdated Show resolved Hide resolved
@ViktorHofer
Copy link
Member

Failures are due to work items being canceled which I pinged dnceng about in another issue.

@ViktorHofer ViktorHofer merged commit 8e3d6cd into dotnet:main Jul 6, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Aug 5, 2021
@Anipik Anipik deleted the performance branch November 10, 2021 18:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants