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

Update README #12

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@ncstang
Contributor

ncstang commented Jan 25, 2015

just fixed a couple of typos.

@mbeijen

This comment has been minimized.

Show comment
Hide comment
@mbeijen

mbeijen Jan 25, 2015

Owner

Well, actually, if I'd merge this now I'd have TWO commits in the repo. Please let me know if you'd want to try again or if I'd just cherry-pick your one commit cf7609a in the repo.

Owner

mbeijen commented Jan 25, 2015

Well, actually, if I'd merge this now I'd have TWO commits in the repo. Please let me know if you'd want to try again or if I'd just cherry-pick your one commit cf7609a in the repo.

@ncstang ncstang closed this Jan 25, 2015

@ncstang

This comment has been minimized.

Show comment
Hide comment
@ncstang

ncstang Jan 25, 2015

Contributor

Michiel,

I went ahead and dropped my branch and reforked your repository. I then
made the changes to the README and created another pull request, so
there is only one commit. Please review and let me know if this one is
good. Thanks.

Sean

On 1/25/2015 10:31 AM, Michiel Beijen wrote:

Well, actually, if I'd merge this now I'd have TWO commits in the
repo. Please let me know if you'd want to try /again/ or if I'd just
cherry-pick your one commit cf7609a
cf7609a
in the repo.


Reply to this email directly or view it on GitHub
#12 (comment).

No virus found in this message.
Checked by AVG - www.avg.com http://www.avg.com
Version: 2013.0.3495 / Virus Database: 4257/8997 - Release Date: 01/25/15


No virus found in this message.
Checked by AVG - www.avg.com
Version: 2013.0.3495 / Virus Database: 4257/8997 - Release Date: 01/25/15

Contributor

ncstang commented Jan 25, 2015

Michiel,

I went ahead and dropped my branch and reforked your repository. I then
made the changes to the README and created another pull request, so
there is only one commit. Please review and let me know if this one is
good. Thanks.

Sean

On 1/25/2015 10:31 AM, Michiel Beijen wrote:

Well, actually, if I'd merge this now I'd have TWO commits in the
repo. Please let me know if you'd want to try /again/ or if I'd just
cherry-pick your one commit cf7609a
cf7609a
in the repo.


Reply to this email directly or view it on GitHub
#12 (comment).

No virus found in this message.
Checked by AVG - www.avg.com http://www.avg.com
Version: 2013.0.3495 / Virus Database: 4257/8997 - Release Date: 01/25/15


No virus found in this message.
Checked by AVG - www.avg.com
Version: 2013.0.3495 / Virus Database: 4257/8997 - Release Date: 01/25/15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment