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 JacksonTian to collaborators #6388

Closed
wants to merge 1 commit into from

Conversation

JacksonTian
Copy link
Contributor

@JacksonTian JacksonTian commented Apr 26, 2016

Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Added myself as a collaborator following the onboarding process.

@jbergstroem
Copy link
Member

LGTM - great to have you on board!

@Trott
Copy link
Member

Trott commented Apr 26, 2016

LGTM. Welcome! Might be a good idea to make the commit message meaningful rather than just "Hi!" just to set a good example for others contributing to the project.

@JacksonTian
Copy link
Contributor Author

thanks @Trott .

Added myself as a collaborator following the onboarding process.
@evanlucas
Copy link
Contributor

LGTM. Welcome aboard

@JacksonTian JacksonTian added the doc Issues and PRs related to the documentations. label Apr 26, 2016
@ChALkeR
Copy link
Member

ChALkeR commented Apr 26, 2016

LGTM

JacksonTian added a commit that referenced this pull request Apr 26, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
@JacksonTian
Copy link
Contributor Author

landed in bf3ffc2

JacksonTian added a commit that referenced this pull request Apr 26, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
jasnell pushed a commit that referenced this pull request Apr 26, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 1, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Jun 24, 2016
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Added myself as a collaborator following the onboarding process.

PR-URL: #6388
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
@JacksonTian JacksonTian deleted the collaborator branch May 4, 2017 07:56
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.

6 participants