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

Added a Changelog #1690

Merged
merged 4 commits into from
Nov 2, 2023
Merged

Added a Changelog #1690

merged 4 commits into from
Nov 2, 2023

Conversation

ADBond
Copy link
Contributor

@ADBond ADBond commented Nov 2, 2023

Type of PR

  • BUG
  • FEAT
  • MAINT
  • DOC

Is your Pull Request linked to an existing Issue or Pull Request?

Adding a changelog as I had previously mooted

Give a brief description for the solution you have provided

Apologies, I accidentally committed the first couple of changes to master, so the 'proper' diff of this PR including those is here - feel free to also make comments on those initial points.
Have updated the PR template + release guide to include the workflow additions.

I think I have reasonably captured our current unreleased changes, but let me know if there are any mistakes / omissions.
I have also back-filled notes for 3.9.8 to help show what the general format looks like, as it was a pretty small release.

Additionally I deliberately decided not to include updates to the docs (or other things such as adding this changelog), as these are not really tied to a particular release (as they go live as soon as they're merged), and so I don't think from a user persepective of 'what functionality is different if I pip install X version' it is really relevant. Happy for any discussion on this point though.

PR Checklist

  • Added documentation for changes
  • Added feature to example notebooks or tutorial (if appropriate)
  • Added tests (if appropriate)
  • Made changes based off the latest version of Splink
  • Run the linter

@ADBond ADBond added documentation Improvements or additions to documentation user experience labels Nov 2, 2023
@ADBond ADBond requested review from RobinL and RossKen November 2, 2023 10:23
Thought it would be useful to have a reference to a full release, and this version was quite a small change
Copy link
Contributor

@RossKen RossKen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a sensible addition. Agreed on keeping it purely to code changes for changes relevant to the API rather than anything on the docs. Thanks for updating the PR template as well! 🥳

@ADBond ADBond merged commit 7e79bb1 into master Nov 2, 2023
@ADBond ADBond deleted the changelog branch November 2, 2023 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants