regro / rever

Releaser of Versions
https://regro.github.io/rever-docs/
BSD 3-Clause "New" or "Revised" License
75 stars 33 forks source link

rever requires xonsh & lazyasd #206

Closed takluyver closed 4 years ago

takluyver commented 4 years ago

So declare the dependencies in setup.py.

Also, regarding the comment about console_scripts: it's fine if you install with pip (pip install . to install locally). It only does the problematic setuptools thing if setuptools does the installation without pip supervising.

CJ-Wright commented 4 years ago

These are the reqs I think: https://github.com/conda-forge/rever-feedstock/blob/master/recipe/meta.yaml#L22

takluyver commented 4 years ago

Thanks, I've copied that list.

Out of interest, do you really see startup times < 0.1 s? Running time rever --help on my machine is consistently giving me about 3 seconds.

CJ-Wright commented 4 years ago

That text was most likely copied from the xonsh setup.py, since startup times are really important for shells.

takluyver commented 4 years ago

ah, that makes more sense.

scopatz commented 4 years ago

Thanks @takluyver!

scopatz commented 4 years ago

And yeah, rever's startup times are pretty slow for some reason. I haven't looked into it, yet though