David Müller
David Müller
OK I just stumbled across this pr [https://github.com/coverlet-coverage/coverlet/pull/1360](https://github.com/coverlet-coverage/coverlet/pull/1360), which basically already covers `ExcludeAssembliesWithoutSources = None`. Should I remove the new parameter name `InstrumentModulesWithoutLocalSources ` now and align it with the...
Hi @MarcoRossignoli , so the issue is that after changing the `sourceRootMappingFilePath`, the `InstrumenterHelper` throws an exception when trying to load the type of the tracker. ```csharp tracker = asm.GetTypes().Single(n...
Not I tried this and it didn't help. Basically every unit test that is using `TestInstrumentationHelper.Run` fails. If I remove all those tests (most of them are in `coverlet.core.tests\Coverage`) I...
The issue is that `Coverlet.Core.Instrumentation.Tracker` isn't injected into the assembly anymore and up to now I couldn't figure out why.
@MelGrubb I think they are all supported in the current version [https://github.com/coverlet-coverage/coverlet/pull/1392/files#diff-be914c2ef40b4434ccb0927c453da5f7f2a4f828d601367f585bd7284349ba85L486-L490](https://github.com/coverlet-coverage/coverlet/pull/1392/files#diff-be914c2ef40b4434ccb0927c453da5f7f2a4f828d601367f585bd7284349ba85L486-L490) but I also could be wrong. At least I'm sure that in the upcoming version we removed this...
Hi @sebastianslutzky, I worked on the PR again and think it fixes the issue. I compiled the source locally and I would be happy if someone could give it try...
@MarcoRossignoli yes I worked on it again and made the integration test run on all platforms now. So this is ready for review/merge. I asked in the issue (https://github.com/coverlet-coverage/coverlet/issues/775#issuecomment-1401168769) if...
Thanks for reporting and thanks for the repro. When enabling the log it can be seen that it is the same issue as #1102. The assembly `Microsoft.Extensions.DependencyInjection.Abstractions, Version=6.0.0.0,` can't be...
@erichiller thanks for reporting this. There is another performance issue reported #1646 for which already a PR exists. Maybe it is the same reason. I let you know once this...
@erichiller The PR I mentioned is now merged and can be consumed with our [nightly](https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md). Maybe you can give it a try. But I guess this here is another issue...