-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
chore(docs): add links to CLI, API, Terraform, and Helm docs #17142
Conversation
Adds snippets to reference the options for managing resources. Related to https://linear.app/prefect/project/strengthen-the-connection-between-prefect-terraform-and-api-docs-1da6564ce040/overview
@@ -0,0 +1,63 @@ | |||
export const home = { |
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.
Here, we define the links in one place and reuse them when we import the snippets. There may be a way to DRY this file up as well, but I haven't found something that works yet with mdx + URLs.
docs/contribute/docs-contribute.mdx
Outdated
## Considerations | ||
|
||
Keep in mind the following when writing documentation. | ||
|
||
### External references | ||
|
||
Prefect resources can be managed in several ways, including through the CLI, UI, Terraform, Helm and API. | ||
|
||
When documenting a resource, consider including external references that describe how to manage the resource in other ways. | ||
Snippets are available to provide these references in a consistent format. | ||
|
||
For example, the [Deployment documentation](/v3/deploy/index) includes a snippet for the Terraform provider: | ||
|
||
```javascript | ||
import { TF } from "/snippets/resource-management/terraform.mdx" | ||
import { deployments } from "/snippets/resource-management/vars.mdx" | ||
|
||
<TF name="deployments" href={deployments.tf} /> | ||
``` | ||
|
||
For more information on how to use snippets, see the [Mintlify documentation](https://mintlify.com/docs/reusable-snippets). |
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.
I added a section here to discuss how to use these snippets. Hopefully this will be a good reminder for future docs so we can keep using these references consistently.
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.
nice!
.github/CONTRIBUTING.md
Outdated
@@ -2,4 +2,4 @@ | |||
|
|||
Thanks for considering contributing to Prefect! | |||
|
|||
To navigate our codebase with confidence, see our [contribution guidelines](https://docs.prefect.io/latest/contributing/overview/). | |||
To navigate our codebase with confidence, see our [contribution guidelines](/contribute/index). |
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.
Looks like this link was broken.
Alrighty, opening up for review. |
Co-authored-by: daniel-prefect <[email protected]>
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.
Thanks @mitchnielsen!
@cicdw @desertaxle @zzstoatzz - looks like I need a review from one of y'all to proceed, thanks in advance 🤝 |
This file is hosted in GitHub, not the docs, so we need the full URL. Co-authored-by: Alex Streed <[email protected]>
Related to https://linear.app/prefect/project/strengthen-the-connection-between-prefect-terraform-and-api-docs-1da6564ce040/overview
Provides references to the CLI, API, Terraform, and Helm docs where applicable to increase awareness of the options available for managing resources.
Adds snippets for references to each resource to keep things DRY.
Checklist
<link to issue>
"mint.json
.