-
Notifications
You must be signed in to change notification settings - Fork 98
Add option for specifying which access levels are included #219
Conversation
…ymbols. This slightly changes the behaviour of the internal data structures which collect information on the symbols (e.g. Interface, Members and Symbol). Now, they contain every symbol, not only the public ones. The decision which symbol to include in the documentation is moved to the generating of the documentation.
@Lukas-Stuehrk Thanks so much for working on this. And apologies for not giving |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, @Lukas-Stuehrk! Just a few, small changes to make on my end. Otherwise, this is ready to go in. Nice work!
🙇 It's always a pleasure to contribute to one of your projects, @mattt. I will sign the commits so the pull request is mergeable. |
5622468
to
7e9ac1d
Compare
Alright, CI is passing so I'll merge this in now. Thanks again for your great work on this, @Lukas-Stuehrk! |
👏 |
This is another try to close #72 - I really want to document my apps.
This slightly changes the behaviour of the internal data structures which collect information on the symbols (e.g.
Interface
,Members
andSymbol
). Now, they contain every symbol, not only the public ones. The decision which symbol to include in the documentation is moved to the generating of the documentation.