Chilipp / autodocsumm

Extending your autodoc API docs with a summary
Apache License 2.0
48 stars 14 forks source link

Remove six #36

Closed Chilipp closed 3 years ago

Chilipp commented 3 years ago

actually we already declared autodocsumm to need python>=3.5 at least, so this is something that has to be removed anyway.

closes #35, ping @sirosen

Chilipp commented 3 years ago

If you want, I can try testing this version against the marshmallow docs (where I ran into the six issue) tomorrow.

yes, sure. That would be helpful, thanks

codecov[bot] commented 3 years ago

Codecov Report

Merging #36 (b919d26) into master (cac91e8) will increase coverage by 0.20%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   85.18%   85.39%   +0.20%     
==========================================
  Files           1        1              
  Lines         270      267       -3     
==========================================
- Hits          230      228       -2     
+ Misses         40       39       -1     
Impacted Files Coverage Δ
autodocsumm/__init__.py 85.39% <66.66%> (+0.20%) :arrow_up:

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 cac91e8...b919d26. Read the comment docs.

sirosen commented 3 years ago

Just FYI, I just reran my doc build with the current master installed via dependency_links=[...] and it worked perfectly. And I see you've been doing some CI/testing work too. Thanks for all your efforts!

Chilipp commented 3 years ago

Great! Thanks for the confirmation @sirosen !

The new release with this fix is available on pypi and conda-forge: 0.2.2