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.

Sign up for GitHub

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

Jump to bottom

refresh infra architecture, updates llms #324

Merged
merged 3 commits into from
Mar 25, 2025
Merged

refresh infra architecture, updates llms #324

merged 3 commits into from
Mar 25, 2025
+62 −14

Conversation

dawnkelly09
Copy link
Collaborator

Description

Refreshes the learn/infra/architecture overview page by:

  • minor copy changes
  • adds Next Steps section

Checklist

  • [x ] Required - I have added a label to this PR 🏷️
  • [ x] Required - I have run my changes through Grammarly
  • [ n/a] If pages have been moved, I have created redirects in the wormhole-mkdocs repo

@dawnkelly09
refresh infra architecture, updates llms
Loading
Loading status checks…
78d0908
@dawnkelly09 dawnkelly09 added A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B0 - Needs Review Pull request is ready for review labels Mar 24, 2025
@dawnkelly09 dawnkelly09 requested a review from eshaben as a code owner March 24, 2025 17:58
@dawnkelly09
updates next steps cards, updates llms
Loading
Loading status checks…
b53ae76
@dawnkelly09 dawnkelly09 requested a review from ilariae March 25, 2025 14:49
ilariae
ilariae approved these changes Mar 25, 2025
View reviewed changes
Copy link
Collaborator

@ilariae ilariae left a comment

Choose a reason for hiding this comment

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

lgtm

eshaben
eshaben approved these changes Mar 25, 2025
View reviewed changes
Copy link
Collaborator

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

LGTM!

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Mar 25, 2025
@martin0995
Merge branch 'main' into dawn/refresh-infra-arch-overview

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
Loading
Loading status checks…
483ffee
@martin0995 martin0995 merged commit 4d2c722 into main Mar 25, 2025
2 checks passed
@martin0995 martin0995 deleted the dawn/refresh-infra-arch-overview branch March 25, 2025 16:28
This was referenced Mar 25, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eshaben eshaben

@ilariae ilariae

@martin0995 martin0995

Assignees
No one assigned
Labels
A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B1 - Ready to Merge Pull request is ready to be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@dawnkelly09 @eshaben @ilariae @martin0995