-
-
Notifications
You must be signed in to change notification settings - Fork 596
Added "reviews" documentation #608
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
Conversation
Nyholm
left a comment
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.
Awesome. Thanks!
|
My english being sometimes uncertain, I would like that some of you check my spelling :) |
Nyholm
left a comment
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.
Minor thing...
I reviewed this more carefully now. I did not find any spelling or phrasing errors.
doc/pull_request/reviews.md
Outdated
| ``` | ||
|
|
||
| ### Dismiss a review | ||
| ##### This does not remove the review but dismisses the (dis)approval status of this one |
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.
Maybe make this bold instead of a heading.
**This does not remove the review but dismisses the (dis)approval status of this one**
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.
ok :)
|
🙌 |
acrobat
left a comment
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.
PR looks good @Geolim4! Thanks!
No description provided.