Skip to content

Added "." on to check_dtype and obj parameter #44446

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

Merged
merged 3 commits into from
Nov 15, 2021

Conversation

Varun270
Copy link
Contributor

  • [] closes #xxxx
  • [] tests added / passed
  • [] Ensure all linting tests pass, see here for how to run them
  • [] whatsnew entry

From #28602
pandas.testing.assert_categorical_equal: Parameter "check_dtype" description should finish with "." pandas.testing.assert_categorical_equal: Parameter "obj" description should finish with "."

@jreback jreback added the Docs label Nov 14, 2021
@jreback jreback added this to the 1.4 milestone Nov 14, 2021
@jreback
Copy link
Contributor

jreback commented Nov 14, 2021

can you merge master


>>> other = pd.DataFrame({'num_legs': [8, 2], 'num_wings': [0, 2]},
>>> other = pd.DataFrame({'num_legs': [8, 3], 'num_wings': [0, 2]},
Copy link
Member

Choose a reason for hiding this comment

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

Why did you change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't change these lines actually I only added the "." at the end of description of "check_dtype" and "obj". These changes are from a different issue that I solved and my PR got merged but I have no idea why these commits got pushed with this PR. You can have a look at this PR for better understanding - https://github.com/pandas-dev/pandas/pull/43820/files

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do let me know what would I have to do in order to get rid of these extra commits.

Copy link
Member

Choose a reason for hiding this comment

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

Simply revert the relevant commit on your branch

Copy link
Member

Choose a reason for hiding this comment

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

something like

git fetch upstream
git rebase upstream/master
git push origin HEAD -f

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done! Kindly review it again.

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.

@MarcoGorelli
Copy link
Member

Thanks @Varun270 - next time I'd advise to make a new branch for each issue you try to solve

@MarcoGorelli MarcoGorelli merged commit a21eae0 into pandas-dev:master Nov 15, 2021
@Varun270
Copy link
Contributor Author

Thanks @Varun270 - next time I'd advise to make a new branch for each issue you try to solve

Noted, I Will keep it in mind!

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.

4 participants