Extend hierarchy calculation for deeply nested pages#250
Extend hierarchy calculation for deeply nested pages#250a-velasco wants to merge 6 commits intocanonical:mainfrom
Conversation
|
/canonical/self-hosted-runners/run-workflows 1d27df7 |
|
It also looks like there have been changed to the discourse change, the ad admin user is no longer there (required for testing), it is now create user ands then promote user. Would you like to fix that? |
Test results for commit e3b2d20Unit tests failed for e3b2d20 ============================= test session starts ============================== tests/unit/action/test_other_actions.py::test__create_directory[dry run mode enabled] PASSED =================================== FAILURES =================================== -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html |
Test results for commit e3b2d20Test coverage for e3b2d20 Static code analysis report |
Overview
A small change to the calculation of page hierarchy to account for navigation tables with large jumps in nesting.
Previously, the difference in whitespace only got evaluated once, which only accounts for cases where the next item is nested one level higher or lower - but not more.
Rationale
This change is needed because the action fails on documentation sets with hierarchy jumps greater than 1. More context + examples in issue #249.
Module Changes
None
Checklist
src-docsurgent,trivial,complex)