Skip to content

Conversation

@MylesBorins
Copy link
Contributor

Useful to have for reference, especially for onboarding

@cjihrig
Copy link
Contributor

cjihrig commented Jan 12, 2016

LGTM

@MylesBorins MylesBorins added the doc Issues and PRs related to the documentations. label Jan 12, 2016
@rvagg
Copy link
Member

rvagg commented Jan 12, 2016

I'd add --simple to that if it's just to get the list of commits

Useful to have for reference, especially for onboarding

PR-URL: nodejs#4636
@MylesBorins
Copy link
Contributor Author

@rvagg updated as requested

@jasnell
Copy link
Member

jasnell commented Jan 12, 2016

LGTM

jasnell pushed a commit that referenced this pull request Jan 12, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
@jasnell
Copy link
Member

jasnell commented Jan 12, 2016

Landed in b6ab6d2

@jasnell jasnell closed this Jan 12, 2016
MylesBorins pushed a commit that referenced this pull request Jan 13, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 14, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
rvagg pushed a commit that referenced this pull request Jan 14, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 14, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 19, 2016
Useful to have for reference, especially for onboarding

PR-URL: #4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Jan 19, 2016
@MylesBorins MylesBorins deleted the branch-diff-example branch January 21, 2016 22:59
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
Useful to have for reference, especially for onboarding

PR-URL: nodejs#4636
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell<jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants