remcohaszing / pywakeonlan

A small python module for wake on lan.
https://pywakeonlan.readthedocs.io
MIT License
281 stars 68 forks source link

Conform to PEP 561 #28

Closed DavidPratt512 closed 1 year ago

DavidPratt512 commented 1 year ago

Refactor the project to have a PEP 561 compliant directory structure. This will allow other dependent packages to use mypy without any hiccups.

Closes #27

remcohaszing commented 1 year ago

I think those @patch decoratores in tests need to be changed from wakeonlan.… to wakeonlan.wakeonlan.… to fix the tests.

codecov[bot] commented 1 year ago

Codecov Report

Merging #28 (9385095) into main (c832b34) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   94.11%   94.11%           
=======================================
  Files           1        1           
  Lines          34       34           
  Branches        7        7           
=======================================
  Hits           32       32           
  Misses          1        1           
  Partials        1        1           
Impacted Files Coverage Δ
wakeonlan/__init__.py 94.11% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

DavidPratt512 commented 1 year ago

So I decided to just move everything to __init__.py. That gets us:

The only downside is that the main function is still exposed. But that's an easy change.

remcohaszing commented 1 year ago

Thank you!

remcohaszing commented 1 year ago

This has been released in version 3.0.0.