Skip to content

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
@estliberitas estliberitas force-pushed the master branch 2 times, most recently from 7da4fd4 to c7066fb Compare April 26, 2016 05:23
@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