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

Make fromkeys() take an Iterable instead of a sequence [REVERTED] #605

Merged
merged 1 commit into from
Oct 14, 2016

Conversation

gvanrossum
Copy link
Member

@gvanrossum gvanrossum commented Oct 14, 2016

(This, too, came up in python/mypy#2254 -- thanks @rwbarton!)

UPDATE: This was rolled back.

@gvanrossum gvanrossum merged commit 66e8d4a into master Oct 14, 2016
@gvanrossum gvanrossum deleted the fromkeys branch October 14, 2016 17:00
@gvanrossum
Copy link
Member Author

Hm... Either this one or the previous one (#604) caused breakage on dict.fromkeys('abcde'). But why?!?!

@gvanrossum
Copy link
Member Author

Oh. It's python/mypy#328

gvanrossum pushed a commit that referenced this pull request Oct 15, 2016
Reason: python/mypy#328

* Revert "Make fromkeys() take an Iterable instead of a sequence. (#605)"

  This reverts commit 66e8d4a.

* Revert "Make dict.fromkeys() a classmethod (which it is). (#604)"

  This reverts commit 26dfcb6.

* Add an explanation for fromkeys() being a staticmethod.
@gvanrossum gvanrossum changed the title Make fromkeys() take an Iterable instead of a sequence. Make fromkeys() take an Iterable instead of a sequence [REVERTED] Mar 15, 2017
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.

1 participant