raimon49 / pip-licenses

Dump the license list of packages installed with pip.
MIT License
298 stars 42 forks source link

Support parsing input parametres from `pyproject.toml` config #200

Closed mbalatsko closed 3 weeks ago

mbalatsko commented 3 weeks ago

This PR adds possibility to set pip-licences input parameters from pyproject.toml file. The change made to be minimally invasive to project logic:

Main motivation: https://github.com/raimon49/pip-licenses/issues/148 and I really love that in modern python packages you could configure all tools in pyproject.toml

raimon49 commented 3 weeks ago

@mbalatsko Thank you! Very interesting suggestion.

Also, your implementation is considerate of existing users and looks very professional.

Toward merging this PR, I think it would be better if the README document has the [tool] table contents of the example TOML file and the specs when loaded.

mbalatsko commented 3 weeks ago

@raimon49 sure, I completely forgot to add it to README, will do!

mbalatsko commented 3 weeks ago

@raimon49 please take a look, should be all there

codecov[bot] commented 3 weeks ago

Codecov Report

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

Project coverage is 99.13%. Comparing base (b3a3d87) to head (ee26a1c).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #200 +/- ## ========================================== + Coverage 99.12% 99.13% +0.01% ========================================== Files 1 1 Lines 457 465 +8 ========================================== + Hits 453 461 +8 Misses 4 4 ```

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

raimon49 commented 3 weeks ago

Just shipped v4.5.0 and we can now use the features implemented in this PR. https://pypi.org/project/pip-licenses/4.5.0/