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

Fixes the build failure due to .NET version change #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

p00j4
Copy link
Contributor

@p00j4 p00j4 commented Apr 14, 2023

Bug in detail: #53

error NETSDK1031: It is not supported to build or publish a self-contained application without specifying a RuntimeIdentifier. You must either specify a RuntimeIdentifier or set SelfContained to false. [/Users/dtsapple/CoveragePublisher/azure-pipelines-coveragepublisher/src/CoveragePublisher.Console/CoveragePublisher.Console.csproj]

Explain fix
Setting SelfContained to False, removes the need of setting the RuntimeIdentifier which is very personal to the user and can't be hardcoded.

Repro steps:
Install .NET6
dotnet build

Or install on .NET 3.1 (like needed for before netcoreapp3.1;net462)

Why it didn't get caught on pipeline
Because the check ran on commit 68874da

Installs .NET7 but as a user, seeing targetFramework .net6, I updated to 6 (from 3.1) and still faced the build failure.
https://github.com/microsoft/azure-pipelines-coveragepublisher/actions/runs/4685895850/jobs/8303415148
image

@p00j4
Copy link
Contributor Author

p00j4 commented Apr 14, 2023

cc: @vinayakmsft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants