Sarah Oslund
Sarah Oslund
I can repro this on 5.0 and 6.0. @Forgind this isn't valid syntax, is it?
Hi all, thanks for the request. We don't have bandwidth to take this on at the moment but tagging @baronfel so the .NET team is aware of this.
@baronfel @KalleOlaviNiemitalo I assumed that you get MSBuild logging for free when using ToolTasks, is that not the case? I have fixed the issue that was blocking the .NET framework...
Sounds good, I have [a branch](https://github.com/microsoft/sbom-tool/compare/sfoslund/netlogging?expand=1) where I seem to have things working when testing e2e but some tests are failing due to a missing MSBuild assembly and the task...
@baronfel do you have any thoughts on this issue? Is this something that needs to be addressed in this repo or one of the .NET repos?
Hi @philipp-naused feel free to make a PR if you would like us to consider this change, I would also love @baronfel to chime in on this before we action...
Looks like we're seeing some CI unit test failures with this change: https://dev.azure.com/mseng/1ES/_build/results?buildId=29662326&view=logs&j=aeb7b77f-2f5c-5c5e-fb92-f326324a83fb&t=947a9a91-a289-5b68-4a98-262ba6071be8
Also @DaveTryon do you know why the 'Check for API changes' commentor check is failing?