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

DOC Update changelog before the 1.2.1 release #25436

Merged
merged 8 commits into from Jan 20, 2023

Conversation

jjerphan
Copy link
Member

@ogrisel and I are preparing the 1.2.1 release.
We think those PRs have to be part of it and have changed the release note accordingly.

We also want to have #25114 #25131 #25129 and #25147 be part of this release, but those do not feature in the changelog (mostly because those are maintenance changes).

What do you think?

@thomasjpfan
Copy link
Member

Here are my thoughts:

@jjerphan
Copy link
Member Author

I agree with what @thomasjpfan suggests. Do you, @ogrisel?

@ogrisel
Copy link
Member

ogrisel commented Jan 20, 2023

Alright!

@jjerphan
Copy link
Member Author

Based on @thomasjpfan's comment, the change-log does not need to be modified. I guess all the issues back-ported from 1.3 to 1.2.X are possible and that the PR is thus mergeable if the CI is green. WDYT?

@jjerphan jjerphan mentioned this pull request Jan 20, 2023
12 tasks
@jjerphan
Copy link
Member Author

FYI I have opened #25442 for releasing 1.2.1, and I am waiting for the current PR (i.e. #25436) to be merged before pursuing the release process.

doc/whats_new/v1.2.rst Outdated Show resolved Hide resolved
Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

LGTM

@thomasjpfan thomasjpfan merged commit 4162ade into scikit-learn:main Jan 20, 2023
@jjerphan jjerphan deleted the doc/retarget-prs-for-1.2.1 branch January 20, 2023 16:44
@jjerphan
Copy link
Member Author

Thanks, @thomasjpfan!

jjerphan added a commit to jjerphan/scikit-learn that referenced this pull request Jan 20, 2023
jjerphan added a commit to jjerphan/scikit-learn that referenced this pull request Jan 23, 2023
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.

None yet

4 participants