pyrogram / tgcrypto

Fast and Portable Cryptography Extension Library for Pyrogram
https://pyrogram.org
GNU Lesser General Public License v3.0
175 stars 41 forks source link

Exclude `tests` from the distributed wheel #33

Open SandaruKasa opened 11 months ago

SandaruKasa commented 11 months ago

From what I can tell, it is discouraged to have a top-level directory named tests distributed alongside with the package: https://wiki.archlinux.org/title/Python_package_guidelines#Test_directory_in_site-package So this PR excludes it from the final .whl being built.

To the best of my understanding, it broke neither the deprecated python3 setup.py test, not the tox, or pytest. However, it might have broken someone's code if it relied on tgcrypto being distributed alongside with the tests. In which case, WTF.

Anyways, this most likely needs a version bump of some sorts, no matter the breakingness of the change.