Skip to content

docs: Add architecture overview guide and more #1306

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
Merged
merged 13 commits into from
Jul 18, 2025
Merged

docs: Add architecture overview guide and more #1306

merged 13 commits into from
Jul 18, 2025

Conversation

vdusek
Copy link
Collaborator

@vdusek vdusek commented Jul 15, 2025

Description

  • Add architecture overview guide.
    • It works more as an introduction-overview guide with links to other guides with more details.
  • Add service locator guide.
  • Add request router guide.
  • Polish existing guides, add missing mermaid diagrams, and code examples.

Issues

Testing

  • Rendered locally

Checklist

  • CI passed

@vdusek vdusek changed the title Docs: Add architecture overview guide and more docs: Add architecture overview guide and more Jul 15, 2025
@github-actions github-actions bot added this to the 119th sprint - Tooling team milestone Jul 15, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Jul 15, 2025
Pijukatel
Pijukatel reviewed Jul 17, 2025
View reviewed changes
Pijukatel
Pijukatel reviewed Jul 17, 2025
View reviewed changes
@vdusek vdusek force-pushed the add-architecture-overview branch from 4b5d165 to 8e3f0f8 Compare July 17, 2025 09:26
Pijukatel
Pijukatel reviewed Jul 17, 2025
View reviewed changes
janbuchar
janbuchar requested changes Jul 17, 2025
View reviewed changes
Copy link
Collaborator

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

Don't get intimidated by those comments - this is legitimately good. Multiple times, I thought "Yeah, but then we need to also mention that ...", only to immediately find out that you thought of that already. Great work!

@vdusek vdusek requested review from Pijukatel and janbuchar July 17, 2025 12:27
janbuchar
janbuchar approved these changes Jul 18, 2025
View reviewed changes
Copy link
Collaborator

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

Nice!

@vdusek vdusek merged commit c560850 into master Jul 18, 2025
35 of 36 checks passed
@vdusek vdusek deleted the add-architecture-overview branch July 18, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@janbuchar janbuchar janbuchar approved these changes

@Mantisus Mantisus Awaiting requested review from Mantisus

@Pijukatel Pijukatel Awaiting requested review from Pijukatel

Assignees

@vdusek vdusek

Labels
t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Milestone
119th sprint - Tooling team
Development

Successfully merging this pull request may close these issues.

Document the crawler & context iheritance Architecture overview
3 participants
@vdusek @janbuchar @Pijukatel
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy