travis ci failing with advent of ipython 6.0.0rc1 about python-fire

[nealmcb]

Looks like 'ipython<6.0' in setup.py will do the trick.

[jtratner]

what's strange is that pip is supposed to ignore it (according to IPython's readme). I'm wondering if the IPython project messed up the trove classifiers or pip did something wrong

[Carreau]

Sorry about that. I'm confused how you get rc1 without using the --pre flag. Or am I missing a configuration option. We'll improve the readme and the docs for downstream projects. Thanks a lot for catching this an warning us upstream.

I'm wondering if the IPython project messed up the trove classifiers or pip did something wrong

The trove classifiers are purely informal. We (the ipython team) had to send patch upstream to Pip, Warehourse, and PyPI, plus modify some peps, in order for the requires_python > 3.3 metadata to be understood by setuptools/pip.. etc. We likely missed something in setuptools.

[jtratner]

Thanks @Carreau - there are a bunch of moving parts here to take care of (and I'm sure it simplifies things for you folks to be python 3.3+ only :) ) An IPython dev suggested a workable way to do conditional requirement for IPython here that should work for us - ipython/ipython#10329 (comment)

…

On Tue, Apr 11, 2017 at 9:33 AM Matthias Bussonnier < ***@***.***> wrote: Sorry about that. I'm confused how you get rc1 without using the --pre flag. Or am I missing a configuration option. We'll improve the readme and the docs for downstream projects. Thanks a lot for catching this an warning us upstream. I'm wondering if the IPython project messed up the trove classifiers or pip did something wrong The trove classifiers are purely informal. We (the ipython team) had to send patch upstream to Pip, Warehourse, and PyPI, plus modify some peps, in order for the requires_python > 3.3 metadata to be understood by setuptools/pip.. etc. We likely missed something in setuptools. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#57 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABhjq9tRVrfVUA7VjqXVJDFZ8aaQwwUSks5ru6tKgaJpZM4M5nQH> .

[Carreau]

Thanks. I'll update the IPython readme with the information Thomas proposal.

(and I'm sure it simplifies things for you folks to be python 3.3+ only :) )

Yes, yes.. I hope to be 3.4+ soon to have the async/await syntax. This is one of the first IPython releases that will have less lines (-1.5k lines) than the previous one. And types annotations are really great even just as documentation.

[dbieber]

Closing since we currently list 'ipython<6.0' as our dependency and travis is happy.
Once we close out #7 this will be even cleaner since we won't list ipython as a required dep at all.

[wookayin]

PR was merged in #58

[dbieber]

#7 is now done as well, so ipython is no longer listed as a dependency.