Skip to content

DOC Directly import label class in example#26876

Merged
OmarManzoor merged 1 commit intoscikit-learn:mainfrom
lucyleeow:doc_lb
Jul 21, 2023
Merged

DOC Directly import label class in example#26876
OmarManzoor merged 1 commit intoscikit-learn:mainfrom
lucyleeow:doc_lb

Conversation

@lucyleeow
Copy link
Copy Markdown
Member

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Nitpick - directly import class in docstring example

Any other comments?

@lucyleeow lucyleeow added the Quick Review For PRs that are quick to review label Jul 21, 2023
@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 960ad78. Link to the linter CI: here

Copy link
Copy Markdown
Contributor

@greyisbetter greyisbetter left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@OmarManzoor OmarManzoor left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @lucyleeow

@OmarManzoor OmarManzoor merged commit b13f69c into scikit-learn:main Jul 21, 2023
@lucyleeow lucyleeow deleted the doc_lb branch July 21, 2023 07:41
punndcoder28 pushed a commit to punndcoder28/scikit-learn that referenced this pull request Jul 29, 2023
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Sep 18, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
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