- Notifications
You must be signed in to change notification settings - Fork294
setup.py: Require ordereddict for Python 2.6#177
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
hoppipolla-critic-bot commentedDec 12, 2014
Critic review:https://critic.hoppipolla.co.uk/r/3483 This is an external review system which you may optionally use for the code review of your pull request. In order to help critic track your changes, please donot make in-place history rewrites (e.g. via |
because html5lib needs it for alpha-sorted attributes when on Python 2.6and it doesn't specify it as a dependency (seehtml5lib/html5lib-python#177)
because html5lib needs it for alpha-sorted attributes when on Python 2.6and it doesn't specify it as a dependency (seehtml5lib/html5lib-python#177)
setup.py: Require ordereddict for Python 2.6
This really shouldn't have been r+'d/merged… This needs a changelog entry, the readme needs to be updated so it's accurate, and the requirements*.txt files need updating. |
Without this, a client who uses the alphabetical attribute sorting feature of
HTMLSerializer
on Python 2.6 will get: