On closer inspection, I'm not sure why we ever need to pass an assembly name into `WarnFailedToLoadAssembly`. On line 41, if we catch an exception, it's during enumeration of the assembly types, not when loading the assemblies. I support switching away from the "ignore the exception" approach, but I don't think reporting a "FailedToLoadAssembly" is necessarily the way to go.