Skip to content

Conversation

@jcrotts
Copy link
Contributor

@jcrotts jcrotts commented Jun 7, 2018

@tirkarthi
Copy link
Member

This PR contains changes from #5179 though it was merged to master. The examples looks like a reasonable addition to me. Can you please try rebasing with latest master?

@jcrotts jcrotts requested a review from a team April 10, 2019 20:56
@jcrotts jcrotts requested review from a team as code owners April 10, 2019 20:56
@gvanrossum
Copy link
Member

Don’t request reviews from the whole world.

@gvanrossum gvanrossum closed this Apr 10, 2019
@jcrotts
Copy link
Contributor Author

jcrotts commented Apr 11, 2019

My Mistake, Not sure where I even requested a review?
Edit: I messed up my rebase the and automerged a bunch of changes, so I think the owners of the different code areas were tagged for review automatically. Sorry for the inconvenience I caused!

@gvanrossum
Copy link
Member

I recommend you open a new PR with just the changes you want. Probably need to update your fork somehow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants