-
-
Notifications
You must be signed in to change notification settings - Fork 26k
[MRG] DOC Replacing "the scikit" with "scikit-learn" #10126
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
Conversation
In Doc, replacing "the scikit" with "scikit-learn"
I prefer scikit-learn be treated as a proper noun: don't put "the" before it. |
Removed "the" from "the scikit-learn" in recent commit. |
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.
@jnothman Some related concerns:
(1)There are some "the scikit" in other folders, e.g., sklearn/, should we also replace them with "scikit-learn"?
(2)There are some "the scikit-learn", should we replace them with "scikit-learn"?
Good point @qinhanmin2014. For (1)
For (2) I could not find quickly looking at the matches a problem with "the scikit-learn". Generally there is a word afterwards like in "the scikit-learn project". I used this command: |
@lesteve Thanks. Agree :) |
@qinhanmin2014 Sure. |
I think we all agree on this one ! @FarahSaeed all the matches I found are in #10126 (comment). You can run the same git grep command to make sure you removed all of them. |
In recent commit, I've extended the replacement ("the scikit" -> "scikit-learn") to all folders. |
@FarahSaeed Seems that there's still another one |
@qinhanmin2014 let me recheck and commit again. |
Replaced "the scikit" with "scikit-learn" in all other places. |
LGTM. I double-checked that there were no remaining "the scikit" occurences with |
Merging without waiting for the CIs, thanks a lot @FarahSaeed ! |
Thank you! :) @lesteve @qinhanmin2014 @jnothman |
What does this implement/fix? Explain your changes.
Fixes #10120
To avoid references to this project as "the scikit" in Doc, replacing "the scikit" with "the scikit-learn"