Michael Gross

> @alex Will [#578 (comment)](https://github.com/nedbat/coveragepy/issues/578#issuecomment-413881957) sill work with your update? @christopherpickering Sorry for jumping in on this, but is this what is still needed to merge this PR? And if...

> @gro1m I'm not sure, that was a long time ago. @christopherpickering I understand that, on the other hand the change is small: the source_path is traversed bottom up from...

> > @gro1m I'm not sure, that was a long time ago. > > @christopherpickering I understand that, on the other hand the change is small: the source_path is traversed...

@aconrad Please let me know what you think of this implementation...

Hi @aconrad I get file system not found errors, can you investigate this. In commit https://github.com/aconrad/pycobertura/pull/137/commits/a5ecd916b818ff9ab8a934b8bce1556cf67e36ef I could fix `test_diff__better_coverage_has_exit_status_of_zero` by adjusting the paths in the xml, but I could...

Strangely, I needed to fix some tests here. But I ask myself how the html File Not Found Error could ever have worked, because these files indeed exist. Can you...

> Ah, we're getting conflicts now, naturally. resolved :)

> Can you update from master? I enabled Github Actions although we don't have the coverage pipeline working for now, will have to do that subsequently. See #139. Yes, done...

@aconrad The tests fail, because the Main class is shown twice. Indeed, the Main class occurs twice in cobertura.xml, so is the test actually correct and should we modify the...

Hi @aconrad Thanks for checking! I think for now it would be best to remove these lines (maybe you could open a separate PR so that we have this well...