Skip to content

Commit b2166a6

Browse files
Merge pull request sendgrid#238 from mptap/add-code-review-contrib
Added Code Review to Contributing.md
2 parents b71ddb8 + 2ecb43d commit b2166a6

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

CONTRIBUTING.md

+6
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ Hello! Thank you for choosing to help contribute to one of the SendGrid open sou
99
- [Testing](#testing)
1010
- [Style Guidelines & Naming Conventions](#style-guidelines-and-naming-conventions)
1111
- [Creating a Pull Request](#creating-a-pull-request)
12+
- [Code Reviews](#code-reviews)
1213

1314
<a name="roadmap"></a>
1415
We use [Milestones](https://github.com/sendgrid/sendgrid-go/milestones) to help define current roadmaps, please feel free to grab an issue from the current milestone. Please indicate that you have begun work on it to avoid collisions. Once a PR is made, community review, comments, suggestions and additional PRs are welcomed and encouraged.
@@ -190,3 +191,8 @@ Please run your code through:
190191
with a clear title and description against the `master` branch. All tests must be passing before we will review the PR.
191192

192193
If you have any additional questions, please feel free to [email](mailto:[email protected]) us or create an issue in this repo.
194+
195+
<a name="code-reviews"></a>
196+
## Code Reviews
197+
198+
If you can, please look at open PRs and review them. Give feedback and help us merge these PRs much faster! If you don't know how, Github has some [great information on how to review a Pull Request](https://help.github.com/articles/about-pull-request-reviews/).

README.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
[![BuildStatus](https://travis-ci.org/sendgrid/sendgrid-go.svg?branch=master)](https://travis-ci.org/sendgrid/sendgrid-go)
44
[![Email Notifications Badge](https://dx.sendgrid.com/badge/go)](https://dx.sendgrid.com/newsletter/go)
55
[![Go Report Card](https://goreportcard.com/badge/github.com/sendgrid/sendgrid-go)](https://goreportcard.com/report/github.com/sendgrid/sendgrid-go)
6-
[![GoDoc](https://godoc.org/github.com/sendgrid/sendgrid-go?status.svg)](https://godoc.org/github.com/sendgrid/sendgrid-go)
6+
[![GoDoc](https://godoc.org/github.com/sendgrid/sendgrid-go?status.svg)](https://godoc.org/github.com/sendgrid/sendgrid-go)
77
[![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](./LICENSE.txt)
88
[![Twitter Follow](https://img.shields.io/twitter/follow/sendgrid.svg?style=social&label=Follow)](https://twitter.com/sendgrid)
99
[![GitHub contributors](https://img.shields.io/github/contributors/sendgrid/sendgrid-go.svg)](https://github.com/sendgrid/sendgrid-go/graphs/contributors)
@@ -242,6 +242,7 @@ Quick links:
242242
- [Bug Reports](https://github.com/sendgrid/sendgrid-go/tree/master/CONTRIBUTING.md#submit-a-bug-report)
243243
- [Sign the CLA to Create a Pull Request](https://github.com/sendgrid/sendgrid-go/tree/master/CONTRIBUTING.md#cla)
244244
- [Improvements to the Codebase](https://github.com/sendgrid/sendgrid-go/tree/master/CONTRIBUTING.md#improvements-to-the-codebase)
245+
- [Review Pull Requests](https://github.com/sendgrid/sendgrid-go/blob/master/CONTRIBUTING.md#code-reviews)
245246

246247
<a name="troubleshooting"></a>
247248
# Troubleshooting

0 commit comments

Comments
 (0)