Skip to content

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