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

Improve read/write-fonts root crate documentation #549

Merged
merged 2 commits into from
Aug 1, 2023
Merged

Conversation

cmyr
Copy link
Member

@cmyr cmyr commented Aug 1, 2023

The module docs in lib.rs are the docs that show up on the main page of docs.rs, and since apparently some people are actually using these crates I think it would be nice if we could make it slightly less painful for them. 💁

Copy link
Member

@dfrg dfrg left a comment

Choose a reason for hiding this comment

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

Left a few notes inline, but this is a substantial improvement and I'm happy to see docs getting some much needed attention!

read-fonts/src/lib.rs Show resolved Hide resolved
write-fonts/src/lib.rs Show resolved Hide resolved
Comment on lines +12 to +14
//! they have a similar structure, where a `tables` module contains a submodule for
//! each supported [table][table-directory], and that module contains items for
//! each table, record, flagset or enum described in the spec. This means that
Copy link
Member

Choose a reason for hiding this comment

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

This is a great description of the code layout. Can we add this to the read-fonts docs as well? I think the duplication is worth it.

This adds a bunch more information to the root crate documentation,
which show up on docs.rs.
These are the docs that appear on docs.rs.
@cmyr cmyr merged commit 025941f into main Aug 1, 2023
7 checks passed
@cmyr cmyr deleted the doc-improvements branch August 1, 2023 18:46
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.

2 participants