Skip to content

Update Readme, to improve example code#248

Merged
tbranyen merged 1 commit intonodegit:masterfrom
nmn:patch-1
Oct 22, 2014
Merged

Update Readme, to improve example code#248
tbranyen merged 1 commit intonodegit:masterfrom
nmn:patch-1

Conversation

@nmn
Copy link
Copy Markdown
Contributor

@nmn nmn commented Oct 22, 2014

Refactored code to use promises chaining for more clarity and less callback hell.

Refactored code to use promises chaining for more clarity and less callback hell.
@tbranyen
Copy link
Copy Markdown
Member

Ha, I was just thinking of doing this myself. Thanks!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like some extra space here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There were some lines already, I left extra spaces before every comment. That's just a style choice. You can make the call.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, if you look at the diff the comment aligned with console. Now it's off by a few. Sorry if I wasn't clear.

@tbranyen
Copy link
Copy Markdown
Member

No worries, I'll fix the whitespace issue. Thanks for the contribution!

tbranyen added a commit that referenced this pull request Oct 22, 2014
Update Readme, to improve example code
@tbranyen tbranyen merged commit 677f4d7 into nodegit:master Oct 22, 2014
@nmn
Copy link
Copy Markdown
Contributor Author

nmn commented Oct 22, 2014

Sorry I missed that.


Sent from Mailbox

On Wed, Oct 22, 2014 at 8:50 PM, Tim Branyen notifications@github.com
wrote:

Merged #248.

Reply to this email directly or view it on GitHub:
#248 (comment)

@tbranyen
Copy link
Copy Markdown
Member

NP it was just a whitespace issue

@nmn nmn deleted the patch-1 branch November 11, 2014 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants