Skip to content

Improve cookbook documentation#738

Merged
swharden merged 20 commits intomasterfrom
api-doc
Feb 5, 2021
Merged

Improve cookbook documentation#738
swharden merged 20 commits intomasterfrom
api-doc

Conversation

@swharden
Copy link
Member

@swharden swharden commented Feb 2, 2021

#727 was a strong start at automated documentation using XML docs as a source.

This PR relies more heavily on XML documentation and less on reflection.

To achieve this, tests were added to fail if XML documentation is lacking in public Plot or Plottable modules

This PR improves support for auto-properties as well (which Reflection sees as methods).

@swharden swharden mentioned this pull request Feb 2, 2021
48 tasks
@swharden swharden marked this pull request as ready for review February 5, 2021 00:35
@swharden
Copy link
Member Author

swharden commented Feb 5, 2021

This PR brought the cookbook forward a long way by adding API information from XML documentation. I can see that the ability to link to a method's URL will be very useful in writing documentation or responding to questions.

However, the cookbook generation code and also the XML/Reflection code were thrown together as an experiment. Although they work somewhat, they have large shortcomings and will be hard to maintain as they are now. Since the cookbook/API documentation style has now been mostly figured out, I think these modules should be re-written with this goal in mind. Templating should probably be separated from page generation too.

@swharden swharden merged commit d1ca60a into master Feb 5, 2021
@swharden swharden deleted the api-doc branch February 5, 2021 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant