Conversation
|
Thanks for submitting your first pull request! You are awesome! 🤗 |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Codecov ReportBase: 91.06% // Head: 91.14% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #88 +/- ##
==========================================
+ Coverage 91.06% 91.14% +0.07%
==========================================
Files 7 7
Lines 739 745 +6
==========================================
+ Hits 673 679 +6
Misses 66 66
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. ☔ View full report at Codecov. |
This fixes an issue where
sphinx-autoupdatedoes not pick up changes to titles in the_toc.yamlfile.To reproduce, use the example project,
sphinx-autobuild --port 8001 . _build/titleattributes to one of the files, egapi.mdYou'll see that the change does not get picked up. The reason is that line 251 of
api.pydoes an equality check on the items in the sitemap:prev_doc = previous[name].Since the title is included in the
Documentclass, the changes in 2. and 3. above trigger a rebuild of theapi.md. But the ToC is injected into theintro.mdfile, which is unchanged both as a file and from the point of view of theget_changedfunction.By including the titles on
FileItems, such changes get picked up byget_changedand the ToC updates correctly.