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

Add technical documentation #44

Merged
merged 3 commits into from
Apr 26, 2023
Merged

Add technical documentation #44

merged 3 commits into from
Apr 26, 2023

Conversation

vdye
Copy link
Collaborator

@vdye vdye commented Apr 24, 2023

Add technical documentation for the bundle server:

  • Commit 1 adds API documentation for git-bundle-web-server.
  • Commit 2 adds a high level architectural diagram of the bundle server with respect to the client & remote, including some internal components.
  • Commit 3 adds flowcharts showing Git's internal algorithm for using a bundle URI (adapted from earlier technical documentation written by @derrickstolee).

@vdye vdye self-assigned this Apr 24, 2023
@vdye vdye force-pushed the vdye/technical-diagrams branch from b7ed924 to 2f6016b Compare April 24, 2023 22:47
Copy link
Collaborator

@ldennington ldennington left a comment

Choose a reason for hiding this comment

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

Great stuff! I think this is going to be so helpful for new contributors when we open source!

Copy link
Collaborator

@derrickstolee derrickstolee left a comment

Choose a reason for hiding this comment

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

I have a few suggestions that might improve things, but these are a really strong documents already.

vdye added 2 commits April 25, 2023 14:34
Add basic API documentation for the web server's endpoints, including a
disclaimer about backward compatibility (or lack thereof). Also, update the
VSCode Markdown settings to use spaces rather than tabs for indentation (to
allow for cleaner sub-tab length alignment here and in future patches).

Signed-off-by: Victoria Dye <[email protected]>
Add a high-level representation of the bundle server architecture, including
descriptions of the major components of a bundle server.

Helped-by: Derrick Stolee <[email protected]>
Signed-off-by: Victoria Dye <[email protected]>
@vdye vdye force-pushed the vdye/technical-diagrams branch 4 times, most recently from f485740 to cbddee1 Compare April 26, 2023 18:15
Add flowcharts documenting how the bundle URI feature in Git downloads from
the bundle server.

Co-authored-by: Derrick Stolee <[email protected]>
Signed-off-by: Victoria Dye <[email protected]>
@vdye vdye force-pushed the vdye/technical-diagrams branch from cbddee1 to d98e9c9 Compare April 26, 2023 18:29
@vdye vdye merged commit b8eaf6a into main Apr 26, 2023
@vdye vdye deleted the vdye/technical-diagrams branch April 26, 2023 19:03
@vdye vdye added this to the v1.0 milestone Apr 26, 2023
vdye added a commit that referenced this pull request Apr 26, 2023
Minor fixups from #44 (for tech docs & VSCode settings)
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.

3 participants