Enter Your Email Address to Watch This Lesson

Your link to unlock this lesson will be sent to this email address.

Unlock this lesson and all 1055 of the free egghead.io lessons, plus get JavaScript content delivered directly to your inbox!



Existing egghead members will not see this. Sign in.

How to Collaborate on a Pull Request on GitHub

2:21 JavaScript lesson by

It’s not often that you’ll get a Pull Request on GitHub just right the first time you try. You generally will iterate with the project maintainer on your solution before your PR gets merged. In this lesson, we’ll explore some of the tools that projects use to manage contributions to open source projects on GitHub.


egghead.io comment guidelines

Avatar
egghead.io

It’s not often that you’ll get a Pull Request on GitHub just right the first time you try. You generally will iterate with the project maintainer on your solution before your PR gets merged. In this lesson, we’ll explore some of the tools that projects use to manage contributions to open source projects on GitHub.

We've patiently waited. Now, the maintainer has gotten back to us.

Before we look at the comments, let's look at some of this information in this poll request. The first thing interesting is this comment from Codecov.io bot. This project uses codecov.io to report coverage. Codecov will comment with the coverage change with each poll request.

It's not totally common to have a bot automatically comment on poll requests, but many major opensource projects on GitHub do have and use bots. The React, Angular and eslint projects all have GitHub bots, for example.

Often, they'll comment to make sure that you've done something, like sign a Contributor License Agreement, or CLA, which pretty much just grants the project owner all the rights to your contributions.

If we scroll down to the bottom here, skipping the comments, we'll see some more information about our poll request. GitHub has a poll request status API, which service is used to provide the information about the poll request.

Here, we have codecov reporting that the poll request itself doesn't meet the standard target for code coverage. Because the poll request lowers coverage for the entire project, it also is reporting that the status for the project coverage doesn't meet the standard for the target code coverage.

This last one is coming from a continuous integration service called Travis CI. Continuous integration is an automatic process that checks your code. It's common for projects to check code style, run tests, and even run a build using a continuous integration service like Travis.

Because the status from Travis CI is green, it means that the code passes all of the checks put into place on Travis CI. We can see that the only thing that we need to update is our code coverage.

Now, let's look at the comments from our project maintainer. We can see that this first comment appears to be inline with the code. This is because GitHub has a feature where you can comment on specific lines in the git-dev.

If we go up here to the files changed, and scroll down to that area of the code, we can see that this comment directly inline with the code. It's a really common practice to comment directly in the dev on GitHub, because it makes it easier for comments to be in the context of the specific code that's being discussed.

You can also comment on the entire PR in the main comments area. If we go back to the conversation tab, scroll down to the bottom, we see that the project maintainer has done this. We can comment at the very bottom here.

Now that we've reviewed the suggestions from the project maintainer, we can go ahead and make our changes, and then, update our PR with those changes.


Featured course:

Building apps with Ionic 2

Building apps with Ionic 2

HEY, QUICK QUESTION!
Joel's Head
Why are we asking?