Skip to content

Drop net461? #1555

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

Closed
pauldotknopf opened this issue Dec 18, 2018 · 3 comments
Closed

Drop net461? #1555

pauldotknopf opened this issue Dec 18, 2018 · 3 comments

Comments

@pauldotknopf
Copy link

It doesn't look like we are using it for any reason, other than the NETDESKTOP define, which doesn't seem needed.

The latest libgit2sharp native binaries are targeting only netstandard, so we wouldn't be able to upgrade.

Can I PR it?

@pauldotknopf
Copy link
Author

See #1544

@pauldotknopf
Copy link
Author

Hmm, it looks like libgit2sharp was already updated! Not sure how it worked, but I'm gonna roll with it.

With that said though, why is the net461 target needed?

@kll
Copy link
Contributor

kll commented Dec 26, 2018

It is considered a good practice to do so, mostly for the benefit of downstream packages consuming GitVersion rather than for GitVersion running by itself.

Using .NET Standard 2.0 from .NET Framework has some issues that were addressed in .NET Framework 4.7.2. You can improve the experience for developers that are still on .NET Framework 4.6.1 - 4.7.1 by offering them a binary that is built for .NET Framework 4.6.1.

More detail in this section of the Open-source library guidance.

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

No branches or pull requests

2 participants