search

executablebooks / sphinx-remove-toctrees

Speed up Sphinx builds by selectively removing toctrees from some pages
MIT License
14 stars 3 forks source link

Element not removed from toctrees #9

Open mscheltienne opened 3 months ago

mscheltienne commented 3 months ago

Describe the bug

In mne-lsl, one of the elements is not removed from the toctree anymore for some reason. No changes beside sphinx and sphinx-remove-toctrees versions.

See the toctree on the left of this page: https://mne.tools/mne-lsl/stable/api/index.html Where mne_lsl.sys_info appears.

Corresponding rST file is similar to the others: https://raw.githubusercontent.com/mne-tools/mne-lsl/main/doc/api/utilities.rst And configuration does remove that unique API folder: https://github.com/mne-tools/mne-lsl/blob/f88dd7bf27b2825be1da0e1d1e65e5d1a19e98d1/doc/conf.py#L289

Pinging @agoose77 who maintained the package recently.

Version used ares available in the Display System Information step of the workflow: https://github.com/mne-tools/mne-lsl/actions/runs/9388354166/job/25853407538

welcome[bot] commented 3 months ago

Thanks for opening your first issue here! Engagement like this is essential for open source projects! :hugs:
If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.
If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).
Welcome to the EBP community! :tada:

mscheltienne commented 3 months ago

And I will add that this problem is due to the changes from @agoose77 Works correctly with commit 28f26a34174a8480763a03f5fbf975c90ae9471f and sphinx 7.3.7.