Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Updated perf testing targets #308

Merged
merged 1 commit into from
Oct 13, 2015
Merged

Updated perf testing targets #308

merged 1 commit into from
Oct 13, 2015

Conversation

ianhays
Copy link

@ianhays ianhays commented Oct 10, 2015

Update to the latest xunit-performance package. Also added a target that will add the new OptimizeForBenchmarks assembly tag to any perf test build. This tag was added in microsoft/xunit-performance#94

@ianhays
Copy link
Author

ianhays commented Oct 10, 2015

@mellinoe

@@ -61,6 +63,11 @@
<AssemblyInfoLines Condition="'$(SkipFrameworkAssemblyMetadata)' != 'true'"
Include="[assembly:System.Reflection.AssemblyMetadata(&quot;%(AssemblyMetadata.Identity)&quot;, &quot;%(AssemblyMetadata.Value)&quot;)]" />
</ItemGroup>

<ItemGroup Condition="'$(RunPerfTestsForProject)' == 'true' and '$(DisableTests)' != 'true'">

This comment was marked as spam.

This comment was marked as spam.

@weshaggard
Copy link
Member

I've renamed Targets=>PackageFiles yesterday so you will need to sync master to resolve your conflicts.

Update to the latest xunit-performance package. Also added a target that will add the new OptimizeForBenchmarks assembly tag to any perf test build. This tag was added in microsoft/xunit-performance#94
@ianhays
Copy link
Author

ianhays commented Oct 13, 2015

@weshaggard done and done

@weshaggard
Copy link
Member

LGTM. Note you will also need to update test-runtime\project.json in CoreFX repo as well.

ianhays pushed a commit that referenced this pull request Oct 13, 2015
Updated perf testing targets
@ianhays ianhays merged commit bc1dadb into dotnet:master Oct 13, 2015
rainersigwald pushed a commit to rainersigwald/buildtools that referenced this pull request Apr 25, 2018
Update WindowsAzure.Storage to latest (dotnet#307)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants