search

astropy / extension-helpers

Helpers to assist with building Python packages with compiled C/Cython extensions
https://extension-helpers.readthedocs.io
BSD 3-Clause "New" or "Revised" License
16 stars 12 forks source link

Remove usage of distutils #34

Closed astrofrog closed 3 years ago

astrofrog commented 3 years ago

Just a quick hack to see if this is possible - see some notes below

Cadair commented 3 years ago

Well that was less controversial than I was expecting.

codecov[bot] commented 3 years ago

Codecov Report

Merging #34 (efa9ec2) into master (16d9f92) will increase coverage by 0.18%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   73.98%   74.16%   +0.18%     
==========================================
  Files           9       10       +1     
  Lines         515      511       -4     
==========================================
- Hits          381      379       -2     
+ Misses        134      132       -2
Impacted Files Coverage Δ
extension_helpers/tests/__init__.py 56.14% <ø> (-2.88%) :arrow_down:
extension_helpers/tests/test_setup_helpers.py 87.87% <ø> (-0.19%) :arrow_down:
extension_helpers/_distutils_helpers.py 93.33% <100.00%> (+2.70%) :arrow_up:
extension_helpers/_openmp_helpers.py 88.03% <100.00%> (+0.85%) :arrow_up:
extension_helpers/_setup_helpers.py 66.32% <100.00%> (-0.35%) :arrow_down:
extension_helpers/tests/test_distutils_helpers.py 100.00% <100.00%> (ø)
extension_helpers/tests/test_openmp_helpers.py 100.00% <100.00%> (ø)

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 16d9f92...efa9ec2. Read the comment docs.