8000 [MRG] Fix whats new link by NicolasHug · Pull Request #15634 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

[MRG] Fix whats new link #15634

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

Merged
merged 2 commits into from
Nov 15, 2019
Merged

Conversation

NicolasHug
Copy link
Member
@NicolasHug NicolasHug commented Nov 15, 2019

The current "What's new in 0.22" button in the index page links to the list of what's news instead of the 0.22's whatsnew

Also fix spelling

@thomasjpfan @adrinjalali @glemaitre

@NicolasHug NicolasHug added this to the 0.22 milestone Nov 15, 2019
@adrinjalali
Copy link
Member

I wouldn't say this is a blocker really.

I'm not sure what the link is supposed to be also. I noticed that it links to the list of whats new entries, but I thought that's by design. If we change that, where do people find the list of the changelogs for different releases?

@NicolasHug
Copy link
Member Author
NicolasHug commented Nov 15, 2019

I'm not sure what the link is supposed to be also.

The button is "what's new in 0.22"

where do people find the list of the changelogs for different releases?

Like before, at the bottom in the "what's new (changelog)" link

I wouldn't say this is a blocker really.

Indeed not a blocker but a no brainer IMHO

@thomasjpfan
Copy link
Member

It use to link to the whats new (with the items) and not a whats_new TOC. With #15288 the link now points to an index page.

Copy link
Member
@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

Right, I thought it's the bottom link. Yeah thanks, it's a nice catch.

@adrinjalali adrinjalali merged commit 3e4ad05 into scikit-learn:master Nov 15, 2019
adrinjalali pushed a commit to adrinjalali/scikit-learn that referenced this pull request Nov 18, 2019
* fix whats new link

* oops fixed bracket
adrinjalali pushed a commit to adrinjalali/scikit-learn that referenced this pull request Nov 18, 2019
* fix whats new link

* oops fixed bracket
adrinjalali pushed a commit that referenced this pull request Nov 19, 2019
* fix whats new link

* oops fixed bracket
panpiort8 pushed a commit to panpiort8/scikit-learn that referenced this pull request Mar 3, 2020
* fix whats new link

* oops fixed bracket
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0