Skip to content

CLN: changed .format to f string in pandas/core/arrays/base.py #30306

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 2 commits into from
Dec 17, 2019

Conversation

Jcole429
Copy link
Contributor

@Jcole429 Jcole429 commented Dec 17, 2019

@WillAyd WillAyd added the Code Style Code style, linting, code_checks label Dec 17, 2019
@WillAyd WillAyd added this to the 1.0 milestone Dec 17, 2019
@WillAyd WillAyd merged commit 0dc356e into pandas-dev:master Dec 17, 2019
@WillAyd
Copy link
Member

WillAyd commented Dec 17, 2019

Sounds good thanks @Jcole429 !

@Jcole429
Copy link
Contributor Author

@WillAyd , What is the best practice for maintaining my fork? Should I delete and make a new fork next time I want to contribute?

This action cannot be undone. This will permanently delete the Jcole429/pandas repository, wiki, issues, and comments, and remove all collaborator associations.

If I delete my fork, will I lose any evidence of collaboration from my profile?

Thank you

@WillAyd
Copy link
Member

WillAyd commented Dec 17, 2019

Set pandas as your upstream if you haven't already (noted in contributor guide) then just always pull updates from upstream/master to your local master. Then just branch locally off master to the branch you want and push that to your fork before making a PR

I wouldn't worry about keeping master on your fork in sync with pandas unless you have another reason to

@Jcole429
Copy link
Contributor Author

Okay, I do have it set as my upstream. I'll do that, thank you!

Just for clarification, if I were to delete my fork, all evidence would be removed from my profile?

@WillAyd
Copy link
Member

WillAyd commented Dec 17, 2019 via email

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.

2 participants