Skip to content
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

Edit readme edits #2

Merged
merged 1 commit into from Apr 24, 2012
Merged

Edit readme edits #2

merged 1 commit into from Apr 24, 2012

Conversation

ajpiano
Copy link

@ajpiano ajpiano commented Apr 23, 2012

Whoops, messed that up in #1, meant to put a PR on your branch, not your master.

@jsvine jsvine merged commit 83a7ee9 into jsvine:edits Apr 24, 2012
@jsvine
Copy link
Owner

jsvine commented Apr 24, 2012

Whoa, sweet, it worked! Pushed back to my branch, and looks like your
change has been automatically added to my original pull request. Thanks for
catching that typo -- definitely not intentional!

On Mon, Apr 23, 2012 at 7:36 PM, adam j. sontag <
reply@reply.github.com

wrote:

Whoops, messed that up in #1, meant to put a PR on your branch, not your
master.

You can merge this Pull Request by running:

git pull https://github.com/ajpiano/learn.jquery.com edit-readme-edits

Or you can view, comment on it, or merge it online at:

#2

-- Commit Summary --

  • Fix typo in readme and continue to tweak local dev setup header

-- File Changes --

M README.md (4)

-- Patch Links --

https://github.com/jsvine/learn.jquery.com/pull/2.patch
https://github.com/jsvine/learn.jquery.com/pull/2.diff


Reply to this email directly or view it on GitHub:
#2

@ajpiano
Copy link
Author

ajpiano commented Apr 24, 2012

Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants