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

ndk: Move library documentation to modules #290

Merged
merged 1 commit into from
Jun 11, 2022

Conversation

MarijnS95
Copy link
Member

Instead of having an outdated overview of modules and their contents in the library documentation, move this to the module documentation header. Rustdoc already provides a clickable link for every module (these manual list didn't include any links), and also includes the first sentence/paragraph from this module documentation.

This also inserts some missing struct documentation and cleans up existing references to link to the NDK docs more often and specify exactly what type it is wrapping, instead of linking to the "rather useless" ffi representation for that NDK type.

@MarijnS95
Copy link
Member Author

With this PR we go from:

image

To:

image

@MarijnS95 MarijnS95 added blocking a release Prevents a new release difficulty: easy Likely easier than most tasks here status: needs review A maintainer must review this code labels Jun 11, 2022
Copy link
Contributor

@msiglreith msiglreith left a comment

Choose a reason for hiding this comment

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

nice!

Instead of having an outdated overview of modules and their contents in
the library documentation, move this to the module documentation header.
Rustdoc already provides a clickable link for every module (these manual
list didn't include _any_ links), and also includes the first
sentence/paragraph from this module documentation.

This also inserts some missing `struct` documentation and cleans up
existing references to link to the NDK docs more often and specify
exactly what type it is wrapping, instead of linking to the "rather
useless" `ffi` representation for that NDK type.
@MarijnS95 MarijnS95 merged commit ac86821 into rust-mobile:master Jun 11, 2022
@MarijnS95 MarijnS95 deleted the doc-improve branch June 11, 2022 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocking a release Prevents a new release difficulty: easy Likely easier than most tasks here status: needs review A maintainer must review this code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants