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

Dual-targeting netcoreapp3.1 #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jehhynes
Copy link

@jehhynes jehhynes commented Nov 4, 2021

No description provided.

@ithielnor
Copy link
Owner

@jehhynes if you'd like to become a maintainer so you can continue updating the nuget, I'll add you.

Since I don't use Harvest any longer it's a bit difficult to maintain.

@jehhynes
Copy link
Author

jehhynes commented Nov 4, 2021 via email

@ithielnor
Copy link
Owner

ithielnor commented Nov 4, 2021

Update the version, then build, then run https://github.com/ithielnor/harvest.net/blob/master/.pack/publish-nuget.ps1 It requires that you have your nuget key in your dropbox like Griddly.

@ithielnor
Copy link
Owner

It looks like you have this setup so it doesn't require separate packages, which is good. But you'll still want to increment the version. I don't think the changes are breaking, so patch version?

@paulirwin
Copy link
Contributor

Not like I would expect a big outrage on this in 2021, but technically going from .NET 4.5 to 4.5.2 is a breaking change. My vote would be for a major version bump.

@jehhynes
Copy link
Author

jehhynes commented Nov 4, 2021

Actually, it already was targeting 4.5.2:

<TargetFrameworkVersion>v4.5.2</TargetFrameworkVersion>

@paulirwin
Copy link
Contributor

@jehhynes Ah sorry, you're right, my eyes caught one of the NuGet packages going from net45 to net452 when scanning the diff.

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.

3 participants