-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
DOC update news for 1.7.1 #31780
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 news for 1.7.1 #31780
Conversation
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.
Should we merge?
Yes please :) |
Maybe we can add "make a separate PR that updates the news" a to do item for the release checklist to avoid the problem of people getting the "you aren't on a stable version" banner after a release? |
Not sure what you mean ? it's already an item in the checklist, see #31762 (comment) |
Ah, maybe the thing to add to it is that we need to do this quite soon after making the release to avoid people reporting that the website is broken |
Well it's a bit tricky and I don't know which step triggers the unwanted banner. The thing is that many steps are ordered, i.e. depend on a previous one, and some steps need to be executed only when we're sure that the release is ready and in a good state. |
I think the problem starts when we update the contents of https://github.com/scikit-learn/scikit-learn.github.io/tree/main/1.7 but don't update the |
No description provided.