basic-reference-assemblies icon indicating copy to clipboard operation
basic-reference-assemblies copied to clipboard

WithReferenceAssemblies Extension Improvement

Open tanveerbadar opened this issue 3 years ago • 4 comments

Hello!

First of, thank you for creating this awesome package. It solved some major headaches for me.

May I suggest an improvement to WithReferenceAssembleis extension method?

Currently, it is implemented as compilation.WithReferences(...) which replaces everything one might have provided during compilation creation. Would it be better to replace it with compilation.AddReferences() which does preserve the existing references?

If the idea sounds good, I can submit a PR to the effect.

tanveerbadar avatar Nov 01 '21 06:11 tanveerbadar

I think there is value in both variations of the methods. Essentially one that replaces the entire set of references and one that adds the references. So rather than "replace" I'd be interested in a PR that augments the existing capability.

I think that AddReferences is a good name for the API you are suggesting.

jaredpar avatar Nov 01 '21 22:11 jaredpar

That's much better! PR on the way in a day or two.

tanveerbadar avatar Nov 02 '21 03:11 tanveerbadar

That's much better! PR on the way in a day or two.

tanveerbadar avatar Nov 02 '21 03:11 tanveerbadar

Has there been any movement on this? I am struggling with similar issues in that I need to be able to add additional references.

RFC1920 avatar Mar 06 '22 21:03 RFC1920