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

Better generated documentation #11

Closed
voider1 opened this issue Feb 28, 2019 · 0 comments · Fixed by #16
Closed

Better generated documentation #11

voider1 opened this issue Feb 28, 2019 · 0 comments · Fixed by #16

Comments

@voider1
Copy link

voider1 commented Feb 28, 2019

When I expose a struct with private fields to my users, they have no way to tell what types to supply. It's also a big blob of text. A bit more spacing and typed would be a big boon for library creators.

chris-morgan added a commit to chris-morgan/rust-typed-builder that referenced this issue Mar 19, 2019
This doesn’t actually solve idanarye#11 (there are still substantial changes
that should be made to the default docs for the `builder()` method), but
it lets you customise how it will appear, and unhide the builder type,
which are both a Good Thing™.

I have written no real tests for this because the system’s not
well-placed for doing that; I’ve just written tests to ensure that it
at least compiles. Yet not testing the documentation generated is not
good.
chris-morgan added a commit to chris-morgan/rust-typed-builder that referenced this issue Mar 19, 2019
This doesn’t actually solve idanarye#11 (there are still substantial changes
that should be made to the default docs for the `builder()` method), but
it lets you customise how it will appear, and unhide the builder type,
which are both a Good Thing™.

I have written no real tests for this because the system’s not
well-placed for doing that; I’ve just written tests to ensure that it
at least compiles. Yet not testing the documentation generated is not
good.
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 a pull request may close this issue.

1 participant