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

[Docs] Sui security first draft #764

Merged
merged 3 commits into from
Mar 12, 2022
Merged

[Docs] Sui security first draft #764

merged 3 commits into from
Mar 12, 2022

Conversation

gdanezis
Copy link
Collaborator

This is the first draft of the sui security doc, providing a high level overview of the guarantees we provide.

@gdanezis gdanezis requested a review from Clay-Mysten March 11, 2022 15:41
@Clay-Mysten Clay-Mysten requested a review from sblackshear March 11, 2022 16:10
Make a slew of edits for grammar and punctuation directly
Copy link
Contributor

@Clay-Mysten Clay-Mysten left a comment

Choose a reason for hiding this comment

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

Hi George,

This is fantastic. I made a ton of small edits directly and included two TODOs where I couldn't make the language clear myself. Please take a look. Otherwise, LGTM. Thanks so much!

@gdanezis
Copy link
Collaborator Author

I addressed the 2 TODOs in the text -- many thanks for going over it. If this is clear feel free to land.

@gdanezis gdanezis added the Type: Documentation Improvements or additions to documentation label Mar 12, 2022
@Clay-Mysten
Copy link
Contributor

Thank you, George!

@Clay-Mysten Clay-Mysten merged commit cb2048c into main Mar 12, 2022
@Clay-Mysten Clay-Mysten deleted the docs-security branch March 12, 2022 19:20
mwtian pushed a commit that referenced this pull request Sep 12, 2022
* chore: bump rust toolchain to 1.63

* fix: makes clippy happy

Fixes the beta build
mwtian pushed a commit to mwtian/sui that referenced this pull request Sep 29, 2022
* chore: bump rust toolchain to 1.63

* fix: makes clippy happy

Fixes the beta build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants