banesullivan / scooby

🐶 🕵️ Great Dane turned Python environment detective
MIT License
47 stars 12 forks source link

use sysconfig for sitepath #83

Closed akaszynski closed 2 years ago

akaszynski commented 2 years ago

Resolves #79 by removing our dependency on distutils.

codecov-commenter commented 2 years ago

Codecov Report

Merging #83 (daf080e) into main (999d9ca) will not change coverage. The diff coverage is 75.00%.

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   87.15%   87.15%           
=======================================
  Files           4        4           
  Lines         366      366           
=======================================
  Hits          319      319           
  Misses         47       47           
prisae commented 2 years ago

Thanks @akaszynski . I'll merge this to subsequently pull it into #85 - but before I will do some benchmarking. sysconfig seems to require a 10th of distutil to load, so we might not even have to lazy-load it any longer.