Enable deterministic build about asynckeyedlock HOT 7 CLOSED

This is implicitly added via DotNet.ReproducibleBuilds. And DotNet.ReproducibleBuilds is a developer dependency (see the PrivateAssets entry of PackageReference Include only for you, the user has no additional dependencies:

<PackageReference Include="DotNet.ReproducibleBuilds" Version="1.1.1" PrivateAssets="All"/>

You only have pros like able to step into code, only small con is a little larger package size as the PDB data with the sourcelink JSON are embedded in the DLL.

from asynckeyedlock.

I will look into the implications of this.

As cons I see adding a dependency for everyone, larger package sizes and I'm not sure if there's any effect at all on performance (do you?)

The pros need to outweigh the cons. Why are you stepping into the source code, do you have an actual problem?

from asynckeyedlock.

I've stumbled upon this, would this work for you?
https://github.com/dotnet/sourcelink

from asynckeyedlock.

Will performance be affected in any way?

from asynckeyedlock.

Are you interested in creating a PR for this so I could create a pre-release version and benchmark performance?

from asynckeyedlock.

Done with #9 :

File size of package is only a bit larger

And when you turn off "Just my code" in VS debug options and try to step into you need to accept the warning

you can step into the code magically

from asynckeyedlock.

Brilliant, thank you!

I released 6.1.1-beta and it's currently being benchmarked against 6.1.0 at https://github.com/MarkCiliaVincenti/AsyncKeyedLockBenchmarks/actions/runs/4025314083/jobs/6918371621

from asynckeyedlock.