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

mailmap updates for upcoming 11.0.0 #9301

Merged
49 commits merged into from Aug 29, 2016
Merged

Conversation

dupont-y
Copy link
Contributor

Please disregard previous one (#9208), and use this one instead.
#9208 will be deleted. I wrongly merged mainstream thinking I was on another branch.

Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
@dupont-y dupont-y changed the title [DNM, WIP] mailmap updates for upcoming 11.0.1 [DNM, WIP] mailmap updates for upcoming 11.0.0 May 30, 2016
@dupont-y
Copy link
Contributor Author

dupont-y commented Jun 6, 2016

Some new entries added.

Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
@@ -529,6 +529,7 @@ Unaffiliated <no@organization.net> Shang Ding <dingshang2013@163.com>
Unaffiliated <no@organization.net> Shanggao Qiu <qiushanggao@qq.com>
Unaffiliated <no@organization.net> Shawn Chen <cxwshawn@gmail.com>
Unaffiliated <no@organization.net> Shawn Edwards <lesser.evil@gmail.com>
Unaffiliated <no@organization.net> shiqi <m13913886148@gmail.com>
Copy link

Choose a reason for hiding this comment

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

👍

Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
@dupont-y
Copy link
Contributor Author

dupont-y commented Jul 4, 2016

AS 11.0.0 is out BUT wasn't a formal release (more a tag than anything else), the process is taking much longer than expected.
I suggest to wait some days (I'll send the last round of mails) and merge this part. Some authors have made modifications to .organisationmap and files are becominc to diverge.

@dupont-y
Copy link
Contributor Author

dupont-y commented Jul 5, 2016

Mails for a390e8e..2680cb4 were sent yesterday evening.

Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
Signed-off-by: Yann Dupont <yann@objoo.org>
@ghost
Copy link

ghost commented Aug 16, 2016

can I merge that now ?

@theanalyst
Copy link
Member

@dachary I guess yann is on vacation (he emailed in the community list a few days ago) and since the pr has been around for sometime I guess we can merge?

@dupont-y
Copy link
Contributor Author

Le 16/08/16 à 16:27, Abhishek L a écrit :

@dachary https://github.com/dachary I guess yann is on vacation (he
emailed in the community list a few days ago) and since the pr has
been around for sometime I guess we can merge?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#9301 (comment), or
mute the thread
https://github.com/notifications/unsubscribe-auth/AGvyZtjpBDxsWqInLh9eIfbsFRZvhzxpks5qgci9gaJpZM4Il6b3.

Well, I'm back now, but not full throttle :) Won't be connected all this
week-end. Next week will be normal.
merge could be a good idea, I could normalize all new changes against it
without having to worry of merge conflicts.

Cheers,

@theanalyst
Copy link
Member

@dachary @dupont-y I guess then it is ok to merge changes until this point, before someone else submits another change in mailmap :) we can have a new pr for any more newer changes

@ghost
Copy link

ghost commented Aug 29, 2016

jenkins test this please (just to get rid of the red)

@ghost ghost changed the title [DNM, WIP] mailmap updates for upcoming 11.0.0 mailmap updates for upcoming 11.0.0 Aug 29, 2016
@ghost
Copy link

ghost commented Aug 29, 2016

@theanalyst you're right. That's a lot of work @dupont-y thanks !

@ghost ghost merged commit 433b028 into ceph:master Aug 29, 2016
@theanalyst
Copy link
Member

Thanks a lot @dupont-y I can imagine a 50 commit pr to be a nice exercise on git rebase ;)

@dupont-y
Copy link
Contributor Author

Le 29/08/16 à 10:59, Loic Dachary a écrit :

@theanalyst https://github.com/theanalyst you're right. That's a
/lot/ of work @dupont-y https://github.com/dupont-y thanks !


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#9301 (comment), or
mute the thread
https://github.com/notifications/unsubscribe-auth/AGvyZmyk6r9v3x0WziK_tZuBdNqu2WVqks5qkp95gaJpZM4Il6b3.

Hi loic, thanks for merging that.
Will now finish that work.

Cheers,

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants