Skip to content

docs: add architecture deep dive#111

Merged
AmitoVrito merged 1 commit into
SynapseKit:mainfrom
DhruvGarg111:docs/architecture-deep-dive
May 30, 2026
Merged

docs: add architecture deep dive#111
AmitoVrito merged 1 commit into
SynapseKit:mainfrom
DhruvGarg111:docs/architecture-deep-dive

Conversation

@DhruvGarg111
Copy link
Copy Markdown
Contributor

Summary

  • Add a long-form Architecture Deep Dive doc with internals, diagrams, and guides
  • Link it from the Architecture overview page
  • Add it to the docs sidebar navigation

Test Plan

  • Not run (docs-only)

Closes #673

@DhruvGarg111
Copy link
Copy Markdown
Contributor Author

@AmitoVrito , looks like the test fail is from your side. Do check

@AmitoVrito
Copy link
Copy Markdown
Contributor

@DhruvGarg111 ,that's the welcome workflow failing, not a docs build or test failure your changes are fine. It's a permissions issue in the CI config on our side, nothing to do with your PR. I will look into it and merge separately

Copy link
Copy Markdown
Contributor

@AmitoVrito AmitoVrito left a comment

Choose a reason for hiding this comment

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

Good write-up, Dhruv. The reading map at the top is a nice touch — makes it easy to jump to the section you actually need rather than reading the whole thing. The CI failure was a bug in the welcome workflow on our end, not your changes. Fixed and pushed to main. Merging now.

@AmitoVrito AmitoVrito merged commit fa7966e into SynapseKit:main May 30, 2026
1 check failed
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