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

Genkit intro doc revamp #760

Merged
merged 10 commits into from
Aug 8, 2024
Merged

Genkit intro doc revamp #760

merged 10 commits into from
Aug 8, 2024

Conversation

chrisraygill
Copy link
Contributor

@chrisraygill chrisraygill commented Aug 7, 2024

Revamping the Genkit intro document.

See staged preview here: https://firebase.devsite.corp.google.com/docs/genkit

docs/index.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@mbleigh mbleigh left a comment

Choose a reason for hiding this comment

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

Only did a quick pass, but I like it better than the status quo for sure.

docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Show resolved Hide resolved
docs/index.md Show resolved Hide resolved
docs/index.md Show resolved Hide resolved
Copy link
Contributor

@i2amsam i2amsam left a comment

Choose a reason for hiding this comment

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

Nice improvement! Left some nitpicks but nothing blocking

@chrisraygill chrisraygill merged commit d5506de into main Aug 8, 2024
4 checks passed
@chrisraygill chrisraygill deleted the chgill-docs-intro-revamp branch August 8, 2024 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants