Skip to content

Conversation

@01-vyom
Copy link
Contributor

@01-vyom 01-vyom commented Mar 11, 2021

Copy link
Member

@MarcoGorelli MarcoGorelli left a comment

Choose a reason for hiding this comment

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

Legend, thanks! Looks good to me pending green

def test_attrs_unknown_column(parser):
with pytest.raises(KeyError, match=("no valid column")):
geom_df.to_xml(attr_cols=["shape", "degreees", "sides"], parser=parser)
geom_df.to_xml(attr_cols=["shape", "degrees", "sides"], parser=parser)
Copy link
Member

Choose a reason for hiding this comment

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

Looks like this is failing a test, might have to choose a different word

@01-vyom
Copy link
Contributor Author

01-vyom commented Mar 11, 2021

Done with all the changes.

@MarcoGorelli MarcoGorelli added the Code Style Code style, linting, code_checks label Mar 11, 2021
@MarcoGorelli MarcoGorelli added this to the 1.3 milestone Mar 11, 2021
@MarcoGorelli
Copy link
Member

Thanks @01-vyom

@MarcoGorelli MarcoGorelli merged commit 082d437 into pandas-dev:master Mar 11, 2021
@01-vyom 01-vyom deleted the pandas-codespell-part3-38802 branch March 11, 2021 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Style Code style, linting, code_checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI/STYLE extend codespell beyond pandas/core

2 participants