Skip to content

DOC_standardized formatting for default values in transformer class.#17505

Merged
thomasjpfan merged 2 commits intoscikit-learn:masterfrom
ezebunandu:doc_default_values_preprocessing_transformer
Jun 6, 2020
Merged

DOC_standardized formatting for default values in transformer class.#17505
thomasjpfan merged 2 commits intoscikit-learn:masterfrom
ezebunandu:doc_default_values_preprocessing_transformer

Conversation

@ezebunandu
Copy link
Copy Markdown
Contributor

[MRG] References Issue #15761

Standardized formatting for default values in the preprocessing.FunctionTransformer class.

cc: @amy12xx

Copy link
Copy Markdown
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.

Thank you for the PR @ezebunandu

The guide recommends using the python basic types (bool).

@ezebunandu
Copy link
Copy Markdown
Contributor Author

[MRG] References Issue #15761

Standardized formatting for default values in the preprocessing.FunctionTransformer class.

cc: @amy12xx

Copy link
Copy Markdown
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! Thank you @amy12xx and @ezebunandu !

@thomasjpfan thomasjpfan merged commit baf8bed into scikit-learn:master Jun 6, 2020
@ezebunandu ezebunandu deleted the doc_default_values_preprocessing_transformer branch June 6, 2020 21:04
@reshamas
Copy link
Copy Markdown
Member

reshamas commented Jun 6, 2020

#DataUmbrella sprint

viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
…cikit-learn#17505)

* DOC_standardized formatting for default values in transformer class.

* DOC_use type bool instead of boolean

Co-authored-by: Sam Ezebunandu <samuel.ezebunandu@outlook.com>
jayzed82 pushed a commit to jayzed82/scikit-learn that referenced this pull request Oct 22, 2020
…cikit-learn#17505)

* DOC_standardized formatting for default values in transformer class.

* DOC_use type bool instead of boolean

Co-authored-by: Sam Ezebunandu <samuel.ezebunandu@outlook.com>
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.

3 participants