Skip to content

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