search

YairHalberstadt / stronginject

compile time dependency injection for .NET
MIT License
845 stars 24 forks source link

Allow single instance structs and null #178

Closed YairHalberstadt closed 2 years ago

YairHalberstadt commented 2 years ago

Improve test coverage

YairHalberstadt commented 2 years ago

@jnm2 In this PR I changed the locking code for single instance dependencies. Obviously I have no expectations for you to review this, but if you do have time to take a quick look at the (non-whitespace) changes to generator tests, that would be super appreciated, seeing how tricky this stuff is to get right.

Thanks!

jnm2 commented 2 years ago

The diff in GeneratorTests.cs looks like what I would do!

YairHalberstadt commented 2 years ago

Amazing thanks!