Skip to content

CLN remove unnecessary trailing commas to get ready for new version of black: _testing -> generic #35949

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Aug 28, 2020
Merged

CLN remove unnecessary trailing commas to get ready for new version of black: _testing -> generic #35949

merged 10 commits into from
Aug 28, 2020

Conversation

MarcoGorelli
Copy link
Member

xref #35925

@MarcoGorelli MarcoGorelli changed the title CLN remove unnecessary trailing commas to get ready for new version of black CLN remove unnecessary trailing commas to get ready for new version of black: _testing -> generic Aug 28, 2020
@MarcoGorelli MarcoGorelli mentioned this pull request Aug 28, 2020
Closed
23 tasks
simonjayhawkins
simonjayhawkins approved these changes Aug 28, 2020
View reviewed changes
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @MarcoGorelli lgtm

@simonjayhawkins simonjayhawkins added Clean Code Style Code style, linting, code_checks labels Aug 28, 2020
@simonjayhawkins simonjayhawkins added this to the 1.2 milestone Aug 28, 2020
charlesdong1991
charlesdong1991 approved these changes Aug 28, 2020
View reviewed changes
Copy link
Contributor

@charlesdong1991 charlesdong1991 left a comment

Choose a reason for hiding this comment

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

thanks!

@charlesdong1991 charlesdong1991 merged commit 47af138 into pandas-dev:master Aug 28, 2020
@MarcoGorelli MarcoGorelli deleted the more-trailing-commas branch August 28, 2020 16:50
AlexKirko pushed a commit to AlexKirko/pandas that referenced this pull request Aug 31, 2020
@MarcoGorelli @AlexKirko
CLN remove unnecessary trailing commas to get ready for new version o…
9d66c13 
…f black: _testing -> generic (pandas-dev#35949)

* pandas/_testing.py

* pandas/core/algorithms.py

* pandas/core/arrays/_mixins.py

* pandas/core/arrays/categorical.py

* pandas/core/arrays/integer.py

* pandas/core/arrays/masked.py

* pandas/core/arrays/numpy_.py

* pandas/core/arrays/period.py

* pandas/core/construction.py

* pandas/core/generic.py
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
@MarcoGorelli
CLN remove unnecessary trailing commas to get ready for new version o…
631f0b2 
…f black: _testing -> generic (pandas-dev#35949)

* pandas/_testing.py

* pandas/core/algorithms.py

* pandas/core/arrays/_mixins.py

* pandas/core/arrays/categorical.py

* pandas/core/arrays/integer.py

* pandas/core/arrays/masked.py

* pandas/core/arrays/numpy_.py

* pandas/core/arrays/period.py

* pandas/core/construction.py

* pandas/core/generic.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@simonjayhawkins simonjayhawkins simonjayhawkins approved these changes

+1 more reviewer

@charlesdong1991 charlesdong1991 charlesdong1991 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Clean Code Style Code style, linting, code_checks

Projects

None yet

Milestone

1.2

Development

Successfully merging this pull request may close these issues.

3 participants

@MarcoGorelli @charlesdong1991 @simonjayhawkins