search

remcohaszing / pywakeonlan

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

Add MANIFEST.in to include README and LICENSE #8

Closed bachp closed 6 years ago

bachp commented 6 years ago

Closes #4

codecov[bot] commented 6 years ago

Codecov Report

Merging #8 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   90.47%   90.47%           
=======================================
  Files           1        1           
  Lines          42       42           
  Branches        6        6           
=======================================
  Hits           38       38           
  Misses          2        2           
  Partials        2        2

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update dfa8a5a...7f0461e. Read the comment docs.

remcohaszing commented 6 years ago

Thank you for your PR.

Because of your PR, I started looking for other ways to include the license. I just don’t like using MANIFEST.in. As you can see in 910ab2776dc5b98210653cbfe20ab54f7346f975, I use setup.cfg to include the license now.

The README is already included in the wheel and egg metadata. In the sdist it’s called README.rst, whereas it’s called DESCRIPTION.rst in the wheel package.