-
Notifications
You must be signed in to change notification settings - Fork 731
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
whitepapers: Update Governor whitepaper #2401
Conversation
tbjump
commented
Feb 17, 2023
- expand section explaining the reasons for defense-in-depth
- add clarity around limit vs. delay
- add FAQ section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure these types of FAQs live in a design doc. They are probably a better fit for documentation as they just restate the more technical explanation in layman's terms
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to @hendrikhofstadt 's comment, I just enabled the wiki feature, perhaps that will be a good home for a plain english description of features.
fb819ed
to
f903f27
Compare
Addressed reviewer feedback: Removed the FAQ section |