executablebooks / sphinx-external-toc

A sphinx extension that allows the site-map to be defined in a single YAML file
https://sphinx-external-toc.readthedocs.io
MIT License
33 stars 18 forks source link

πŸ‘Œ IMPROVE: Changed document identification #51

Closed chrisjsewell closed 3 years ago

chrisjsewell commented 3 years ago

Move the comparison of sitemaps and identification of changed documents to SiteMap.get_changed, and add tests for this method.

codecov[bot] commented 3 years ago

Codecov Report

Merging #51 (0389a59) into main (091d23a) will increase coverage by 0.29%. The diff coverage is 87.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   90.46%   90.75%   +0.29%     
==========================================
  Files           6        6              
  Lines         650      660      +10     
==========================================
+ Hits          588      599      +11     
+ Misses         62       61       -1     
Flag Coverage Ξ”
pytests 90.75% <87.50%> (+0.29%) :arrow_up:

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

Impacted Files Coverage Ξ”
sphinx_external_toc/events.py 90.12% <50.00%> (+0.48%) :arrow_up:
sphinx_external_toc/api.py 93.25% <100.00%> (+1.05%) :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 091d23a...0389a59. Read the comment docs.