.NET Standard

[alhardy]

PR for #17

[AppVeyorBot]

❌ Build HdrHistogram.NET 1.20.0-koomuqfb failed (commit a960190ed5 by @alhardy)

[AppVeyorBot]

❌ Build HdrHistogram.NET 1.20.0-vrhmmlkx failed (commit f40e362d72 by @alhardy)

[AppVeyorBot]

❌ Build HdrHistogram.NET 1.20.0-mitbstyt failed (commit 82c554c568 by @alhardy)

[alhardy]

nuget.exe removed here 7cfe633

and added to gitignore here 3f85777

[AppVeyorBot]

❌ Build HdrHistogram.NET 1.20.0-ptwbfvvp failed (commit c1d970643d by @alhardy)

[alhardy]

default.ps1 is more of a psake convention, there is a build.cmd in the solution root which calls out to that.

[LeeCampbell]

Nice work @alhardy.
A few things need to get fixed before this gets merged.

Can we squish the commits. i.e.

• seems no point having the nuget.exe committed to the repo, then removed in the same PR.
• I would rather the build scripts not know about AppVeyor, and have their SemVer passed in as an argument
• merge commits instead of rebase

So it seems that there are 2 legitimate commits of the 7 in the PR. I can pick up this work and lean heavily on your good work if you are not up for it.

I will also need to change the AppVeyor builds as you have moved the build scripts, so this currently wont even build.

[alhardy]

@LeeCampbell yeah the appveyor build's broken due to the build changes I've made.

I won't get a chance to make those changes for a couple of weeks, I've added you as a collaborator to my fork if you're happy to pick up the requested changes beforehand.

[ghuntley]

@LeeCampbell the tooling has changed so much you would be better off burning this PR to the ground and doing it again. Happy to 1:1 and run you through how RxUI now does xtargeting/nuget creation. See ReactiveUI/ReactiveUI repo if you prefer self-service.

[LeeCampbell]

Thanks @alhardy finally closed by #58