search

coverlet-coverage / coverlet

Cross platform code coverage for .NET
MIT License
2.98k stars 386 forks source link

Rework build definition #1524

Closed lg2de closed 5 months ago

lg2de commented 1 year ago

Makes build definition less complex, hopefully...

The old SDKs (3.1, 5.0) are not used anymore. The solution is using netstandard, net4xx, and net6.0, so SDK 6.0 is required for build. When global.json is used, we should install the version specified there. Further, net7 is used explicitly in the integration tests and must be installed separately. Please note that currently only 7.0.203 is working fine. Newer versions causes compiler warnings/errors. This should be fixed separately.

Triggered and scheduled builds were using less shared definition than possible. This causes broken nightly build in the past.

The tests for MVCRAZOR and ASPNET were not executed on build server because of the wrong project settings. I fixed that and activated option to fail if this happens again.

lg2de commented 11 months ago

@daveMueller , @MarcoRossignoli , please note that this fixes EXECUTING of some integration tests.

Bertk commented 5 months ago

This PR was closed because it is outdated.

lg2de commented 5 months ago

What means outdated? It was actual when I created the PR. It's a pity that it just stayed there.

Bertk commented 5 months ago

Sorry, the sequence of PRs are random but this PR was not rebased and is somehow obsolete.