sphinxcontrib-towncrier icon indicating copy to clipboard operation
sphinxcontrib-towncrier copied to clipboard

Update build-system dependencies

Open dvzrv opened this issue 2 years ago • 4 comments
trafficstars

pyproject.toml: Update setuptools-scm to >= 7.1.0 to drop the requirement for setuptools-scm-git-archive (since the functionality is included since 7.0.0). Update requirement for setuptools accordingly.

dvzrv avatar Jan 09 '23 21:01 dvzrv

This will probably need to wait until I drop Python 3.6.

webknjaz avatar Jan 09 '23 22:01 webknjaz

Codecov Report

Merging #80 (2fcfcb2) into master (729b600) will not change coverage. The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   64.63%   64.63%           
=======================================
  Files          14       14           
  Lines         246      246           
  Branches       30       30           
=======================================
  Hits          159      159           
  Misses         86       86           
  Partials        1        1           

codecov[bot] avatar Jan 09 '23 22:01 codecov[bot]

This will probably need to wait until I drop Python 3.6.

I didn't even notice you still support 3.6. Is there a reason for it? After all, upstream already EOL'd it :)

dvzrv avatar Jan 10 '23 09:01 dvzrv

I usually don't rush to drop support if there's no compelling reason to do so. Especially, in small projects, where it takes more effort than just leaving it as is. Plus, in this project it is much more important to work on adding the integration tests to improve the coverage and be able to stop shipping only alpha versions, plus use a better trove classifier for stability.

FWIW, from the Python packaging perspective, I declare minimum versions with the software features needed and then, it works with the newer versions too. Bumping the lower bound doesn't seem justified right now. But thanks for bringing this to my attention — I should really test building dists with the lowest supported deps, including the runtime/interpreter so such a breaking change would be noticeable in CI.

Re:patch — I think, that it could be acceptable now if instead of replacing the entries it made use of env markers to have them as is for Python 3.6 and only add the updated deps for Python 3.7+ (just 7.0). Note that setuptools-scm 7+ no longer declares the toml extra so it's not needed anymore. Oh, and one more thing that is missing from this patch is updating .git_archival.txt — the newer plugin version makes use of newer Git features and requires a newer template there — you can take that file from this plugin's Git tree itself.

webknjaz avatar Jan 10 '23 16:01 webknjaz