Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some improvements to the documentation #47

Merged
merged 2 commits into from
Jun 25, 2019
Merged

Conversation

aperezdc
Copy link
Contributor

Change a bit how HotDoc is invoked, and start documenting the public API. Note that this is a shy start, and it would be great to have more people contribute with documentation—but at least this is a start!

See #37


You can see how the generated documentation looks at https://people.igalia.com/aperez/libwpe-doc-sample/

docs/loader.md Outdated Show resolved Hide resolved
aperezdc added 2 commits June 25, 2019 00:32
This makes it possible for HotDoc to find the generated version.h
header.
Manually group the deprecated versioning symbols in a general version
section, and start documenting some of the headers.

Additionally, make HotDoc pick the generated headers, and do some small
tweaks to its invocation.
@aperezdc aperezdc force-pushed the aperezdc/hotdoc-love branch from c3ac839 to 18a81f7 Compare June 24, 2019 22:11
@aperezdc aperezdc requested a review from thiblahute June 25, 2019 12:46
Copy link
Contributor

@thiblahute thiblahute left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

include/wpe/version.h.cmake Show resolved Hide resolved
@aperezdc aperezdc merged commit 7f1bfe3 into master Jun 25, 2019
@aperezdc
Copy link
Contributor Author

Merged, given that our resident HotDoc expert agreed with the changes 😉

@thiblahute
Copy link
Contributor

Do you have the macros documented now?

@aperezdc
Copy link
Contributor Author

@thiblahute Not yet, I'll make a new PR with a few more things documented in the next days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants