Skip to content

Conversation

@AdarshSantoria
Copy link
Contributor

Mentioning #3401 #3404 #3413 in HISTORY.md

Copy link
Member

@rcurtin rcurtin left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together---I have some suggestions on how we can keep the changelog shorter and easier to read for users. 👍

AdarshSantoria and others added 2 commits February 13, 2023 01:20
Co-authored-by: Ryan Curtin <[email protected]>
Co-authored-by: Ryan Curtin <[email protected]>
Copy link
Member

@rcurtin rcurtin left a comment

Choose a reason for hiding this comment

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

Looks good to me; let's wait until #3404 and #3413 are merged to merge this. 👍 Thanks!

Copy link

@mlpack-bot mlpack-bot bot left a comment

Choose a reason for hiding this comment

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

Second approval provided automatically after 24 hours. 👍

@rcurtin rcurtin merged commit c6c54aa into mlpack:master Feb 17, 2023
@AdarshSantoria AdarshSantoria deleted the history branch February 22, 2023 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants