Skip to content

Make docs more LLM friendly #29566

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

Merged

Conversation

C00ldudeNoonan
Copy link
Contributor

Summary & Motivation

Added description tag to .md and .mdx pages across the docs to assist in SEO and making the docs more LLM friendly.

How I Tested These Changes

Built docs locally and checked metadata on webpage and description is in head.

Copy link
Contributor

@neverett neverett left a comment

Choose a reason for hiding this comment

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

Sending over the first batch of edits! Once you've incorporated these edits, I can re-review and do another 50 or so.

I also noticed that the space between the front matter and first line of content has been removed -- that should be added back for all files.

@C00ldudeNoonan C00ldudeNoonan marked this pull request as ready for review April 28, 2025 15:49
@C00ldudeNoonan C00ldudeNoonan requested a review from a team as a code owner April 28, 2025 15:49
@C00ldudeNoonan C00ldudeNoonan force-pushed the anoonan/doc-1101-generate-descriptions-for-non-api-docs branch from 5b5157b to 3dce112 Compare April 28, 2025 16:09
…oc-1101-generate-descriptions-for-non-api-docs
@C00ldudeNoonan C00ldudeNoonan requested a review from neverett April 28, 2025 16:19
Copy link

github-actions bot commented Apr 28, 2025

Deploy preview for dagster-docs ready!

Preview available at https://dagster-docs-81nc87og1-elementl.vercel.app
https://anoonan-doc-1101-generate-descriptions-for-non-api-docs.archive.dagster-docs.io

Direct link to changed pages:

Copy link
Contributor

@neverett neverett left a comment

Choose a reason for hiding this comment

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

Reviewed another chunk of pages

neverett added 24 commits May 1, 2025 13:17
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
… sections

Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Signed-off-by: nikki everett <nikki@dagsterlabs.com>
@neverett neverett marked this pull request as ready for review May 6, 2025 19:03
@neverett neverett self-requested a review May 6, 2025 19:04
@neverett neverett merged commit 9d2b754 into master May 6, 2025
7 of 8 checks passed
@neverett neverett deleted the anoonan/doc-1101-generate-descriptions-for-non-api-docs branch May 6, 2025 19:14
neverett added a commit that referenced this pull request May 12, 2025
## Summary & Motivation
Added description tag to .md and .mdx pages across the docs to assist in
SEO and making the docs more LLM friendly.

## How I Tested These Changes
Built docs locally and checked metadata on webpage and description is in
head.

---------

Signed-off-by: nikki everett <nikki@dagsterlabs.com>
Co-authored-by: Nikki Everett <neverett@users.noreply.github.com>
Co-authored-by: nikki everett <nikki@dagsterlabs.com>
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