BUG: changed io.wb.get_countries URL by clarkfitzg · Pull Request #6008 · pandas-dev/pandas (original) (raw)

Your PRs showed that you were substantially invested in writing code and you made
a truely collosal mess of your PR history at the time, so it seemed right to push you
towards learning the tools. At this point, you probably know git better then me.

If you'll recall, I did not teach you git (except on specific points you asked about).
What I suggested is that if you're not comfortable with git that you leave it to the
maintainers to sort out, which I did. It really is less work often, and less stressful
on well-intentioned contributors who are otherwise made to run the git gauntlet.

The amount of impromptu git tutorials that span more comments then the actual discussion
on the substance of the PR is really high here. Not every typo fix PR mandates that the submitter
have an in-depth understanding of the git merge-model. Leave'em alone.

If someone really screws up, admonish. if someone starts submitting lots of good PRs
marred only by bad git fu, by all means walk them through it and reciprocate their efforts.
Otherwise, pull-fix-push-close (while maintaining authorship). I've often done that in the past.

You do this because you're truely nice, generous people who like to share knowledge
and help out others. But sometimes, it's just too much.

And I sincerly hope you've learned to shrug off my admonishments. It's just a thing.