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

doc: add romankl to collaborators #3725

Closed
wants to merge 1 commit into from
Closed

Conversation

r-52
Copy link
Contributor

@r-52 r-52 commented Nov 9, 2015

No description provided.

@MylesBorins
Copy link
Contributor

LGTM

3 similar comments
@evanlucas
Copy link
Contributor

LGTM

@Fishrock123
Copy link
Contributor

LGTM

@Trott
Copy link
Member

Trott commented Nov 9, 2015

LGTM

@Fishrock123
Copy link
Contributor

Aside:

@romankl you should probably add yourself to the [mailmap] so that your old git info of Roman Klauke <[email protected]> maps to your new email.

@Fishrock123 Fishrock123 added the doc Issues and PRs related to the documentations. label Nov 9, 2015
@saghul
Copy link
Member

saghul commented Nov 9, 2015

LGTM

@jbergstroem
Copy link
Member

🎉 !

@thefourtheye
Copy link
Contributor

This one-liner has a conflict 😮

@r-52
Copy link
Contributor Author

r-52 commented Nov 10, 2015

@Fishrock123 should I include the update in this pr or wait till the next "update AUTHORs" pr ?

@jbergstroem
Copy link
Member

@romankl rebase this branch onto latest master and force push the commit to this branch (to make sure).

@Fishrock123
Copy link
Contributor

@romankl no that can be done in a separate commit. :)

r-52 added a commit that referenced this pull request Nov 10, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
@r-52
Copy link
Contributor Author

r-52 commented Nov 10, 2015

Thank you!! Landed in 5235143

@r-52 r-52 closed this Nov 10, 2015
@r-52 r-52 deleted the add-to-collab branch November 10, 2015 18:52
r-52 added a commit that referenced this pull request Nov 11, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
@Fishrock123 Fishrock123 mentioned this pull request Nov 11, 2015
r-52 added a commit that referenced this pull request Nov 17, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
@MylesBorins
Copy link
Contributor

landed in v4.x-staging in a49dd89

r-52 added a commit that referenced this pull request Dec 4, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
@jasnell jasnell mentioned this pull request Dec 17, 2015
r-52 added a commit that referenced this pull request Dec 17, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
r-52 added a commit that referenced this pull request Dec 23, 2015
PR-URL: #3725
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Saúl Ibarra Corretgé <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
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.

9 participants