search

alchemistry / alchemtest

the simple alchemistry test set
https://alchemtest.readthedocs.io
BSD 3-Clause "New" or "Revised" License
8 stars 12 forks source link

NEP29 #76

Closed xiki-tempula closed 2 years ago

xiki-tempula commented 2 years ago

Remove support for py<3.7.

codecov[bot] commented 2 years ago

Codecov Report

Merging #76 (8a97305) into master (c574471) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   99.34%   99.34%           
=======================================
  Files          11       11           
  Lines         152      152           
  Branches       19       18    -1     
=======================================
  Hits          151      151           
  Misses          1        1
Flag Coverage Δ
unittests 99.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

xiki-tempula commented 2 years ago

@orbeckst @DrDomenicoMarson is using pathlib for the path related operations in https://github.com/alchemistry/alchemtest/pull/71, so it is good to deprecate the versions that do not support pathlib.

I wonder when you say documentation, what do you mean? I have searched the 2.7 and cannot find more hits.

orbeckst commented 2 years ago

Add a note to the docs that states that we follow NEP29, pretty much the same as in the alchemlyb docs that I linked in #77

xiki-tempula commented 2 years ago

@orbeckst Right. I have added the secion to index.rst saying that we will adopt NEP29 from 0.8.0