-
Notifications
You must be signed in to change notification settings - Fork 566
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
Sanitizes Code Reviews README and adds information about metrics #172
Sanitizes Code Reviews README and adds information about metrics #172
Conversation
…ME and adds information about CR metrics.
Could you link the issues in the "Linked issues" section of the PR pls? |
1 similar comment
Could you link the issues in the "Linked issues" section of the PR pls? |
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.
Could you link the issues in the "Linked issues" section of the PR pls?
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.
Looks great.
I just have one concern, there is a PR being reviewed currently on the same readme about size guidance.
Perhaps you should coordinate so that the changes here (or areas that you already checked and feel are ok) don't get overwritten by that as I think a lot of them are in the same sections.
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
Closing this PR as it has become convoluted, and starting fresh. |
Addresses issues #168 and #120