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

Updates to documentation #139

Merged
merged 3 commits into from
Feb 3, 2025
Merged

Updates to documentation #139

merged 3 commits into from
Feb 3, 2025

Conversation

DGaffney
Copy link
Contributor

Description

No technical changes here, just re-pinning the documentation to our current understanding!

Reference: CV2-5887

How has this been tested?

No tests here

Are there any external dependencies?

NA

Have you considered secure coding practices when writing this code?

NA

@DGaffney DGaffney changed the title Updates to documentaiton Updates to documentation Jan 28, 2025
Copy link
Contributor

@skyemeedan skyemeedan left a comment

Choose a reason for hiding this comment

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

Great, I think the diagrams better aligned with the how it works now

@DGaffney DGaffney merged commit 70ad579 into master Feb 3, 2025
2 checks passed
@DGaffney DGaffney deleted the cv2-5887-presto-docs branch February 3, 2025 21:57
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