Geoffrey Fairchild
Geoffrey Fairchild
I know my issue above turned out not to be related to this issue, but I figured out why I needed to add this: ``` import logging logging.root.setLevel(logging.NOTSET) ``` to...
I've followed the instructions so far: ``` docker pull quay.io/pypa/manylinux1_x86_64 docker run -it -v $(pwd):/io quay.io/pypa/manylinux1_x86_64 ``` But I don't see a `wheelhouse` directory that you guys seem to have...
But the script you posted above expects `*.whl` files to already exist in `/wheelhouse` when logging into the Docker container: ```bash for whl in wheelhouse/*.whl; do auditwheel repair "$whl" -w...
Ok, I think I figured it out. From https://github.com/pypa/python-manylinux-demo/blob/master/travis/build-wheels.sh, this is key: ``` for PYBIN in /opt/python/*/bin; do "${PYBIN}/pip" wheel /io/ -w wheelhouse/ done ```
@xzy3, I apologize for how long it's taken for me to review this relatively simple PR. Life has been pretty crazy for a while now. One concern I have with...
Hm, I think that might be better. I really don't want to worry about maintaining two separate copies of the same code. Does this also mean that because it's two...
Makes sense. I think I'm going to hold off on merging this for now. While the `__init__.pyi` won't be changing unless I modify the root library somehow, `setup.cfg` will change...
> I made the updates to setup.cfg Thanks for these changes! I'm still curious of there's a way to consolidate `setup.cfg` and the root [`setup.py`](https://github.com/gfairchild/pyxDamerauLevenshtein/blob/master/setup.py) to eliminate redundancy altogether.
> Not that I particularly found. That's what about 90% of this thread has been about. I might have missed something though. Yeah, and I may just be too picky...
> I do not understand why you even want to ship the type stubs as a separate project. Why not just include them in the normal installation? My main hesitation...