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

Update project maintainers #60

Merged
merged 2 commits into from
Mar 18, 2016
Merged

Update project maintainers #60

merged 2 commits into from
Mar 18, 2016

Conversation

vdemeester
Copy link
Contributor

This updates project maintainer for : docker/engine-api and docker/docker.

@moxiegirl @thaJeztah Are the changes for Curators and Docs maintainers normal ? 👼

Signed-off-by: Vincent Demeester vincent@sbr.pm

@thaJeztah
Copy link
Member

@vdemeester yes, the "Curators" and "Docs maintainers" sections aren't generated properly by the script; the script hasn't been modified yet to make it work with multiple projects/repos having a "Docs maintainers" or "Curators" section, so the last repository containing one will overwrite the previous ones (instead of merging the people in those).

Last time I updated, I just edited that manually, but we should update the script so that it works 😄

@vdemeester
Copy link
Contributor Author

@thaJeztah Ok 😛 I'll take a look at the script too then 😉

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@vdemeester
Copy link
Contributor Author

@thaJeztah updated the code and re-ran it 👼

@thaJeztah
Copy link
Member

nice! LGTM

thaJeztah added a commit that referenced this pull request Mar 18, 2016
@thaJeztah thaJeztah merged commit 9dd45b4 into docker:master Mar 18, 2016
@vdemeester vdemeester deleted the update-maintainers branch March 18, 2016 18:31
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.

2 participants