pydoctor icon indicating copy to clipboard operation
pydoctor copied to clipboard

Visit all AST branches

Open tristanlatr opened this issue 2 years ago • 2 comments

#589 fixes the issue regarding the ast.If and ast.Try nodes but it doesn’t for the the ast.For nodes. So anything that is in the else block of a for loop, it won’t be processed. It not a huge issue since for loops typically resides inside function body, and else block does not contain any documentable entries, but it’s still a regression.

Maybe I should just add the to the PR #589

tristanlatr avatar Jun 02 '22 20:06 tristanlatr

Ok, so after some investigations, this behaviour is not a new behaviour and it's been like that forever in pydoctor (more than 16 years).

So the PR #589 is good but totally misguided, because it does not fix an issue, it provides a new feature.

tristanlatr avatar Jul 13 '22 22:07 tristanlatr