search

disqus / nydus

Nydus is a Python toolkit for managing database connections and routing operations, primarily for Redis
Apache License 2.0
380 stars 38 forks source link

Problems with test requirements #33

Closed offlinehacker closed 10 years ago

offlinehacker commented 10 years ago

Your tests requirements require 'Django>=1.2,< 1.5', but sentry, which uses this package requires Django>=1.5.5. Can you do something like Django<=1.5.5, so we can run automated tests for both packages? Thanks!

dcramer commented 10 years ago

The test requires require <1.5, though we can update it to allow Django 1.5 since it's supported

That said, there's no reason you need to run Nydus' tests unless you're developing on Nydus

offlinehacker commented 10 years ago

Well yes, but in out distro, we tend to run python test for every package, but we have conflicts in this package. Looks like we will have to disable tests or solve this differently. Python package managment is such a mess...

dcramer commented 10 years ago

Fixed in f4e80cba582227bb60f5f7c2d089e4fd5614f3fe

offlinehacker commented 10 years ago

Tnx, we will enable tests in next release of nydus. Just for info, i'm atomatically generating python packages for nix package manager: http://nixos.org/nix/