-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Defibrillator badge to tomtheisen #1241
Conversation
How do I indicate my acceptance? Just by saying so here? Here's me, saying so. |
@tomtheisen do you want your real name listed? if so what is it? |
List me as "Tom Theisen" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the name
@tomtheisen: I know this is different than other places. Thank you for the feedback, we're always iterating to improve. To accept, you just hit the checkmark in the second set of checkboxes. |
ps. Now we just wait for everyone to approve. |
Defibrillator badge to tomtheisen
@mention the contributor: @tomtheisen
Recommendation to:
See #1239
As the one mentioned, I would like to:
within 30 days, if you have not indicated which option you are taking one of the following will happen:
Note: All committers must approve via review before merging, the disapproving committer can simply close the PR.