8000 Add Steve and Mary as docs maintainers by shykes · Pull Request #10866 · moby/moby · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Steve and Mary as docs maintainers #10866

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

Merged
merged 1 commit into from
Feb 19, 2015
Merged

Conversation

shykes
Copy link
Contributor
@shykes shykes commented Feb 18, 2015

No description provided.

Signed-off-by: Solomon Hykes <solomon@docker.com>
@GordonTheTurtle
Copy link

Can you please sign your commits following these rules:

https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work

The easiest way to do this is to amend the last commit:

$ git clone -b "shykes-new-docs-maintainers" git@github.com:docker/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

@shykes shykes force-pushed the shykes-new-docs-maintainers branch from 392e3c3 to ed1b491 Compare February 18, 2015 03:31
@shykes
Copy link
Contributor Author
shykes commented Feb 18, 2015

Damn you gordon!

@shykes shykes force-pushed the shykes-new-docs-maintainers branch from ed1b491 to 0d41c6a Compare February 18, 2015 03:33
@crosbymichael
Copy link
Contributor

LGTM


[people.mary]
Name = "Mary Anthony"
Email = "mary.anthony@docker.com"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't there be a GitHub handle?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also; the list should be in alphabetical order 😄

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thaJeztah I will fix in followup PR

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jfrazelle Want me to do that? or would it be faster if you did it? :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I already had a patch fixing something in it with tomlv too, so I just
added it to that, pinged you on it too :)

On Thu, Feb 19, 2015 at 11:45 AM, Sebastiaan van Stijn <
notifications@github.com> wrote:

In MAINTAINERS
#10866 (comment):

@@ -588,3 +590,7 @@ made through a pull request.
Name = "Vishnu Kannan"
Email = "vishnuk@google.com"
GitHub = "vishh"
+

@jfrazelle https://github.com/jfrazelle Want me to do that? or would it
be faster if you did it? :)


Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10866/files#r25018358.

@SvenDowideit
Copy link
Contributor

LGTM - @fredlf @jamtur01

(y, need to add @moxiegirl's GH handle)

mmm, need to find the docs subsystem rules - I presume we don't want to go back to needing 3 LGTM's

@thaJeztah
Copy link
Member

@SvenDowideit You can create a DOCS-BOT and add it as maintainer. Everytime you LGTM, it'll automatically repeat you 😸

@fredlf
Copy link
Contributor
fredlf commented Feb 19, 2015

lgtm

@jamtur01
Copy link
Contributor

Awesome! LGTM.

jessfraz pushed a commit that referenced this pull request Feb 19, 2015
Add Steve and Mary as docs maintainers
@jessfraz jessfraz merged commit 009efd0 into master Feb 19, 2015
@jessfraz jessfraz deleted the shykes-new-docs-maintainers branch February 20, 2015 00:14
SvenDowideit pushed a commit to SvenDowideit/docker that referenced this pull request Feb 26, 2015
Add Steve and Mary as docs maintainers
(cherry picked from commit 009efd0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants
0