Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build output does not indicate build failures #4370

Open
MichaelSimons opened this issue May 3, 2024 · 2 comments
Open

Build output does not indicate build failures #4370

MichaelSimons opened this issue May 3, 2024 · 2 comments
Labels
area-product-experience Improvements in the end-user's product experience

Comments

@MichaelSimons
Copy link
Member

I was building in a devcontainer via ./build.sh -sb -bl. There was a build failure that I noticed in the build output scrolling by when watching the build yet nothing called out the build error when the build finished.

  [14:54:31.60] Building razor...done
  New artifact(s) after building razor:
    -> Microsoft.AspNetCore.Mvc.Razor.Extensions.Tooling.Internal
    -> Microsoft.CodeAnalysis.Razor.Tooling.Internal
    -> Microsoft.NET.Sdk.Razor.SourceGenerators.Transport
  [14:57:39.58] Building fsharp...done
  New artifact(s) after building fsharp:
    -> Microsoft.FSharp.Compiler
[root@codespaces-bb5a33 dotnet]# echo $?
1

Scrolling up in the build output I found the build error:

    /vmr/src/vstest/src/Microsoft.TestPlatform.CoreUtilities/GlobalSuppressions.cs(1,1): error IDE0073: A source file contains a header that does not match the required text (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0073) [/vmr/src/vstest/src/Microsoft.TestPlatform.CoreUtilities/Microsoft.TestPlatform.CoreUtilities.csproj::TargetFramework=net9.0]
##[error]/vmr/src/vstest/src/Microsoft.TestPlatform.CoreUtilities/GlobalSuppressions.cs(1,1): error IDE0073: (NETCORE_ENGINEERING_TELEMETRY=Build) A source file contains a header that does not match the required text (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0073)

Is this something related to the parallel build infra?

Copy link

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

@MichaelSimons MichaelSimons added area-product-experience Improvements in the end-user's product experience and removed untriaged labels May 9, 2024
@MichaelSimons
Copy link
Member Author

[Triage] The expectation in this case is to at a minimum have text that clearly indicates the build failed. Optimally the build errors would be enumerated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-product-experience Improvements in the end-user's product experience
Projects
Status: 9.0
Development

No branches or pull requests

1 participant