Louis Sautier

Hi, it would be nice to have those for the same reason as https://github.com/mbr/visitor/pull/2

Hi, the args module isn't used, it shouldn't be listed here.

Add git tags

3

Hello, It looks like GitHub is missing all tags for releases, could you please add them?

Hi, Could you consider including the examples in PyPI distributions to make it easier for us to install them?

Possibly unused requirements: future and six (for testing)

1

Hi, I think that the `future` requirement is not necessary, all the imports that the project uses seem to be listed [here](https://docs.python.org/2/library/__future__.html) and are available for Python >= 2.6. Tests...

Tests sometimes fail with PyPy3

9

Hi, It looks like tests sometimes fail when running PyPy3 (pypy-7.3.7-final): ``` test/test_plyvel.py::test_threading PASSEDpypy3: /build/leveldb-yUlUFD/leveldb-1.22/db/version_set.cc:779: leveldb::VersionSet::~VersionSet(): Assertion `dummy_versions_.next_ == &dummy_versions_' failed. Fatal Python error: Aborted Stack (most recent call first,...

Why isn't nosudo the default?

1

Hi, I noticed that on Proxmox all users can run zfs commands. Is that also the case for other systems? Are there any reasons to avoid running the commands as...

Hi, It took me a long time to understand why `setuptools_scm.get_version(fallback_root="..")` didn't work inside a subfolder of a PyPI tarball. Would it be possible to mention `fallback_root` somewhere in README?...

pytest-runner listed in setup_requires

1

Hello, I think it is a mistake to list pytest-runner in setup_requires. Sure, it's required for testing (and already present in test_requires), but there is no reason to require non-developers...

Add MANIFEST.in to include demo and docs in PyPi tarballs, fixes #425

2

Hi, Distributions often rely on PyPI tarballs to build docs. The current PyPI tarballs do not contain them. The sdist file that I obtain with the MANIFEST.in file also contains...