Adi Roiban

+1 for having only public API in objects.inv by default objects.inv primary usage is for linking to external API, and you should only link to public external API. ---------- we...

> have pydoctor consider public methods from a private class that are inherited by a public class (SynchronousTestCase) to be public? +1 - the method is on a public class,...

> Meaning **rendered** private API documentation (HTML), right? Yes. The rendered API documentation doesn't make sense. ----------- > Not exactly: names with underscores are not a problem for the tools,...

In general I guess that before running pydoctor on a python code, you also run compileall or pyflakes or other checkers. I am not sure if pydoctor should handle everything...

What do you say about the support regex? Regex syntax is ugly but should be powerful enough to do the job done, and regex is pretty common and well understood.

> because we can't just hide **.test, this would maybe hide some crucial API elements. So I went with the following way: So if `**.test` doens't work. My hope was...

Regarding the explicit names > --privacy="HIDDEN:re:twisted\.(words|web|runner|python|etc)\.test" Which fullName contains `.test.` in it and it should not be hidden? Yes. We might need to have 2 rules, but I guess it...

To get a report if a test is not executed as part of the CI...hence there is no coverage for it. Create codecov.yaml ``` codecov: require_ci_to_pass: yes notify: # We...

I think that the best practice is to have something like this ``` _factory = RandomFactory() """ Cleanup as we don't want direct usage of the factory. @hidden """ secureRandom...

Sorry ... I have updated the example to have the docstrings on the initial assignment :) I am happy to review a PR in which `del` is used for API...