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

Proposal: move to CHANGELOG-based release process #881

Merged
merged 9 commits into from
Jan 23, 2022

Conversation

baronfel
Copy link
Contributor

This simplifies the build script with the following changes:

  • move to Ionide.KeepAChangelog + migrate old release notes to changelog
  • purge a lot of changelog-related code from build.fsx, now we only need it for some copy/paste archive folder management
  • move creation of the github release + package push to the github actions release pipeline
    • these are now pretty simple scripts that are still easily runnable locally, just copy + paste the exact command
  • remove no longer used targets from build.fsx, the primary custom stuff we do now is a) fsliblog namespacing, and b) tool package munging
  • update paket/FAKE to .net 6 versions
  • update to .net 6 SDK (but not TFM)

@baronfel baronfel force-pushed the new-release-process branch 2 times, most recently from 8f72965 to 08a55b7 Compare January 17, 2022 04:52
@baronfel
Copy link
Contributor Author

@Krzysztof-Cieslak any thoughts or concerns here? the flow would be:

  • push tag from local
  • ???
  • profit

though I could see making a FAKE task to put some safety around the tag (making sure the changelog version doesn't already exist as a tag, etc)

@baronfel baronfel force-pushed the new-release-process branch from be5c6a2 to e103c39 Compare January 22, 2022 23:46
@baronfel
Copy link
Contributor Author

I made a FAKE target called "Promote" that automates promoting the changelog's Unreleased segment to a new version, with some logic around what that version should be. it then commits the changelog and makes the matching tag.

@baronfel
Copy link
Contributor Author

I'm pretty happy with this, and it's in line with things that @Krzysztof-Cieslak and I have talked about in the past, so I'm going to merge it.

@baronfel baronfel merged commit 5aabe5a into ionide:main Jan 23, 2022
@baronfel baronfel deleted the new-release-process branch January 23, 2022 19:35
@baronfel
Copy link
Contributor Author

successfully released 0.50.0 using this method (after a tiny snafu with an expired nuget key) and it all worked great!

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.

1 participant