-
-
Notifications
You must be signed in to change notification settings - Fork 726
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
GH3572: Only build TargetFrameworks netcoreapp3.1, net5.0 & net6.0 #3574
Conversation
devlead
commented
Oct 10, 2021
•
edited by pascalberger
Loading
edited by pascalberger
- fixes Only build for TargetFrameworks netcoreapp3.1, net5.0 and net6.0 #3572
- fixes Stop shipping Cake runner for .NET Framework and Cake runner for .NET Core #3579
- fixes Stop shipping Cake.Portable Chocolatey package and Cake Homebrew formulae #3581
8f1d69f
to
3a8312e
Compare
Removing OpenCover makes sense, but are we planning on replacing this with something like Coverlet? https://github.com/coverlet-coverage/coverlet |
3a8312e
to
84b94d3
Compare
Probably, what's there now has been disabled for a long time, but yes we can always add something later. Removing the current code meant removing dependency on Mono, which is quite a simplification for our build pipeline. |
993cdf9
to
8439829
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once this is merged Azure Pipeline need to be updated as well to get rid of the no longer required installation steps of now unsupported .NET versions
Yes, same for Bitrise. |
7ef21c7
to
78c0825
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Added a couple of comments.
da4fa02
to
e4310e2
Compare
e4310e2
to
670bdfa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM