On Sat, 01 Oct 2016 10:04:42 -0400, Johan Huldtgren
<[email protected]> wrote:

> hello,
> 
> On 2016-10-01 09:32, Daniel Jakots wrote:
> > 
> > # tests are available, but need pytest-runner plus dep's (which
> > aren't in # tree) at build time; if adding these, remove
> > patch-setup_py NO_TEST =       Yes
> > 
> > but they removed the needs of pytest-runner [0], and other depends
> > seem to be in the tree, have you look at it?
> > 
> > [0]:
> > https://github.com/jpadilla/pyjwt/commit/acc30b90666f2edb891a674bbb1727114776421a
> >   
> 
> however it's still in setup.py
> 
> --
> 
> tests_require = [
>      'pytest==2.7.3',
>      'pytest-cov',
>      'pytest-runner',
> ]
> 
> --
> 
> so I'm not sure it's been completely eliminated, also tests seem to
> fail (all this said I'm not a python person and might be completely
> misreading this I just use this as a dependency for beets)

Indeed, the commit is a bit weird.

Anyway, I committed this diff and the one for py-oauthlib, thanks!

Cheers,
Daniel

Reply via email to