stack-overflow-copy-paste icon indicating copy to clipboard operation
stack-overflow-copy-paste copied to clipboard

First open source pull request!

Open bcarson opened this issue 9 years ago • 6 comments

I think I've got it this time, please let me know if I need to correct anything. Thanks again for your help Kent!

bcarson avatar May 09 '16 15:05 bcarson

Fantastic! Thanks for the pull request! Let me add a few inline comments for you. I have some feedback :-)

kentcdodds avatar May 09 '16 15:05 kentcdodds

Looking pretty good. When you've completed what I've noted, could you follow the instructions on how to squash your commits? When you do, you'll get a chance to update your commit message which must follow our conventions

Let me know if you have any trouble! Enjoy learning :D Thanks!

luck-good-im-rooting-for-you-12XDYvMJNcmLgQ

kentcdodds avatar May 09 '16 15:05 kentcdodds

Aye aye captain :) I have to work today, but I’ll incorporate your comments and resubmit soon.

From: "Kent C. Dodds" [email protected] Reply-To: eggheadio-github/stack-overflow-copy-paste [email protected] Date: Monday, May 9, 2016 at 10:24 AM To: eggheadio-github/stack-overflow-copy-paste [email protected] Cc: Bonnie Brennan [email protected], Author [email protected] Subject: Re: [eggheadio-github/stack-overflow-copy-paste] First open source pull request! (#26)

Looking pretty good. When you've completed what I've noted, could you follow the instructions on how to squash your commits? When you do, you'll get a chance to update your commit message which must follow our conventions

Let me know if you have any trouble! Enjoy learning :D Thanks!

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub

bcarson avatar May 09 '16 16:05 bcarson

ping :)

kentcdodds avatar Sep 13 '16 16:09 kentcdodds

I squashed my commits but I have merge conflicts, not sure what I did wrong there.

bcarson avatar Sep 13 '16 23:09 bcarson

It's quite possible that your local master branch is behind this project's master. You'll want to set this repo as an upstream and have your local master track upstream/master as demonstrated in this video.

kentcdodds avatar Sep 14 '16 03:09 kentcdodds