-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add documentation fields and generate READMEs #9
Conversation
base64uuid/Cargo.toml
Outdated
@@ -1,6 +1,8 @@ | |||
[package] | |||
name = "base64uuid" | |||
description = "Type for representing base64url-encoded UUIDs" | |||
documentation = "https://docs.rs/base64uuid/latest/base64uuid/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont need this as docs.rs replaces this with the actual docs.rs url on publishing to crates.io
fiberplane-models/Cargo.toml
Outdated
@@ -1,6 +1,8 @@ | |||
[package] | |||
name = "fiberplane-models" | |||
description = "Core models used across Fiberplane" | |||
documentation = "https://docs.rs/fiberplane-models/latest/fiberplane-models/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
@@ -1,6 +1,8 @@ | |||
[package] | |||
name = "fiberplane-provider-runtime" | |||
description = "Wasmer runtime for Fiberplane Providers" | |||
documentation = "https://docs.rs/fiberplane-provider-runtime/latest/fiberplane-provider-runtime/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again, i think u get it now so ill stop
Description
Adds
documentation
andreadme
fields to all the published crates. Also puts the bulk of the crate READMEs inside of the crate'ssrc/lib.rs
, so that it will also be displayed prominently ondocs.rs
. I added some READMEs where they were still missing, and fixed some links in the process.Only crate exempted for now is the API client, because the generator has no method yet to generate the given fields, or to include a README as the directory gets wiped every time (@mellowagain).
Fixes FP-2997
Checklist
Please make sure all of these are checked before merging. Please leave items
you think are non-applicable in the list, but use strike-through (
~~
) toindicate they don't apply.
[ ] The changes have been tested to be backwards compatible.[ ] The OpenAPI schema and generated client have been updated.[ ] PR for API is ready and reviewed: (please link)[ ] PR for Studio is ready and reviewed: (please link)[ ] PR for CLI is ready and reviewed: (please link)[ ] PR for Proxy is ready and reviewed: (please link)[ ] The CHANGELOG(s) are updated.When adding new
fiberplane-models
module:[ ] PR for fp-openapi-rust-gen is ready and reviewed: (please link)When adding new operation types:
[ ] PR for fiberplane-ot is ready and reviewed: (please link)After merging, please merge related PRs ASAP, so others don't get blocked.