Skip to content
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

tuples don't have a pop() method. The same change has been made upstream... #693

Closed
wants to merge 1 commit into from

Conversation

pmav99
Copy link

@pmav99 pmav99 commented Feb 1, 2012

@mdboom
Copy link
Member

mdboom commented Feb 1, 2012

+1. Can you rebase this on v1.1.x maintenance -- the bug appears to exist there, too.

@pmav99
Copy link
Author

pmav99 commented Feb 2, 2012

Sorry but I am not a git user, and I can't work out how I should do that. For good or for bad I have erased the patch-1 branch. Currently I have 2 commits on my fork. One on master and one on v1.1.x (I cherry picked the commit from master to v1.1.x so they are the same).

It seems that I can't add more commits on this pull request. And when I try to make a new pull request I can't find how I can add commits from two different branches. Could you provide some guidance?

ps. The time I spent on this is disproportionally large for something so trivial. Perhaps it would be easier if you just took the time to make the change and push it...

@mdboom mdboom closed this Feb 3, 2012
@mdboom
Copy link
Member

mdboom commented Feb 3, 2012

Closed by 97c58d2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants