-
-
Notifications
You must be signed in to change notification settings - Fork 470
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
Fix typos in design.md #199
base: main
Are you sure you want to change the base?
Conversation
c008ee2
to
cfccfc2
Compare
20d1729
to
2464a55
Compare
28b2820
to
7a8c14a
Compare
803547d
to
8f0acbf
Compare
this PR seems good |
f797b76
to
a44de89
Compare
5ca2bb9
to
708ad63
Compare
4652763
to
230976a
Compare
192efa1
to
a1711b9
Compare
ccf2cda
to
7d91674
Compare
bf2c440
to
df9e997
Compare
1ca7b17
to
d6db850
Compare
There's another typo, seems like this PR hasn't been covered yet. |
I don't really care too much about typos in the comments and these supplemental documents that are intended to be read only by developers. |
@rui314 Is there a reason not merge this? When people find typos, they are willing try to help the project. Just merging them would be the easiest for you, wouldn't it? |
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.
LGTM.
Signed-off-by: Christian Sattler [email protected]