-
Notifications
You must be signed in to change notification settings - Fork 74
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
Release 6.1.2 #192
Release 6.1.2 #192
Conversation
FYI I turned off merge commits just now (as well as wikis and projects) as https://github.com/fsprojects/Argu/commits/master/ seems unnecessarily messy (IME you either need to police commits and go around requesting rebasing, or just stick to squash merges). Open to debate though. |
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 would like to see 174 at some point but that that should be grounds for a feature bump and 187 can join it I guess?
Hm seeing lots of build work, ( If this was me, I'd be tempted to merge my two PRs and mint a 7.0.0-rc.1, then turn it RC after we prove it's correct wrt jumping to source and the deterministic build flags at https://nuget.info/packages/Argu etc It'd be good to have the release number in the title of these PRs too (personally in general I tend to use MinVer and put tags on commits, but then I don't have the FAKE skills to do it the hard way!) |
Agree probably not breaking anything (was avoiding thinking about it!) |
* Add 6.1.2 to release notes. * Add dependabot yml * Add release job. * Add NuGet key * Add permissions * Add 116 entry.
* Add 6.1.2 to release notes. * Add dependabot yml * Add release job. * Add NuGet key * Add permissions * Add 116 entry.
This is an attempt to perform a release via a manual dispatch action.
I'd like to start with just being able to run the release, we can improve some stuff in the future afterwards.
resolves #166
fixes #116
fixes #159