search

mahmoudimus / nose-timer

A timer plugin for nosetests (how much time does every test take?)
MIT License
126 stars 33 forks source link

Drop ordereddict requirement #84

Closed jakirkham closed 7 years ago

jakirkham commented 7 years ago

As Python 2.7 is the minimum Python supported, every version of Python should have ordereddict preincluded in the standard library one way or another. So we can drop this dependency and just handle the differences between Python 2 and Python 3.

jakirkham commented 7 years ago

Would add that ordereddict appears to have unintentionally become a hard dependency of nose-timer. IOW if ordereddict is not present, nosetests fails to recognize the --with-timer option. The change here fixes that issue.

skudriashev commented 7 years ago

@jakirkham thanks for your PR.

mahmoudimus commented 7 years ago

👍