Assembly name comparison

[Sten-V]

Is it possible that this call to Contains(...) instead should be a call to Equals(...), or is this intentional?

RazorLight/src/RazorLight/Compilation/DefaultMetadataReferenceManager.cs

Line 105 in 22381db

if (excludedAssemblies.Any(s => s.Contains(assemblyRef.Name))) { continue; }

[toddams]

excludedAssemblies is a collection, assemblyRef.Name is a string. This is intentional

[Sten-V]

Yes, but the Contains(...) is not performed directly on that collection, it done on every string in that collection the way it looks now.
And because it is a Contains(...) comparison between two strings it's quite possible to exclude more assemblies than the consuming code intended.

[toddams]

Sorry, I actually overlooked the point. It's indeed weird, I'll double-check