Julius2342 / pyvlx

PyVLX - controling VELUX windows with Python via KLF 200
GNU Lesser General Public License v3.0
73 stars 26 forks source link

Update setup.py, exclude 'test' #370

Closed onkelbeh closed 6 months ago

onkelbeh commented 6 months ago

Otherwise:

* Messages for package dev-python/pyvlx-0.2.21:

 * The following unexpected files/directories were found top-level
 * in the site-packages directory:
 *
 *   /usr/lib/python3.11/site-packages/test
 *
 * This is most likely a bug in the build system.  More information
 * can be found in the Python Guide:
 * https://projects.gentoo.org/python/guide/qawarn.html#stray-top-level-files-in-site-packages
 * ERROR: dev-python/pyvlx-0.2.21::HomeAssistantRepository failed (install phase):
 *   Failing install because of stray top-level files in site-packages

I hope you don't mind I included your component for Home Assistant Gentoo Overlay. I do not use this component myself. But, during compilation test, the installation test script threw an error, because setup.py tries to install a package called tests at top level.

My change only adds a generic exclusion to find_packages().

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4ae18e3) 80.00% compared to head (3f08bf0) 80.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #370 +/- ## ======================================= Coverage 80.00% 80.00% ======================================= Files 77 77 Lines 3560 3560 ======================================= Hits 2848 2848 Misses 712 712 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.