Update the list of developers and contributors about Yasuhiro Kimura.

[yasu]

Summary

It corresponds to the 2nd step of the procedure described in section
7.1 of Committer's Guide.

Approved by: meta(mentor), ygy(mentor)

[ygy]

Looks good to me, approved. Two nits on the commit message:

• There is a space before "(mentor)", e.g. meta (mentor) instead of meta(mentor)
• Don't forget the "Differential Revision:" entry :)

[yasu]

Adject commit message.

• Add 'Differential Revision:'
• Remove lines added by 'arc diff'.

[ygy]

• The first line of the commit message is not a full sentence, hence you can remove the ending period.
• Just a reminder that the "Approved by:" etc are tab-separated, e.g. Approved by:(tab) ygy (mentor)
• Again, there should be a space before "(mentor)", e.g. meta (mentor) instead of meta(mentor).

[yasu]

In D32133#725199, @ygy wrote:

Looks good to me, approved.

Thanks.

Two nits on the commit message:

• There is a space before "(mentor)", e.g. meta (mentor) instead of meta(mentor)

I'll fix it.

• Don't forget the "Differential Revision:" entry :)

I have added it :-).

BTW I noticed that commit messages are modified by arc command when I create new review with arc diff.
Is there any way to prevent it?

[ygy]

In D32133#725208, @yasu wrote:

BTW I noticed that commit messages are modified by arc command when I create new review with arc diff.
Is there any way to prevent it?

I am not an expert on arc, and I don't use it at all - maybe @meta can weigh in here. That said, you have my approval to commit (just change it locally, preview and push it without arc) once you addressed the items that I have mentioned:

In D32133#725207, @ygy wrote:

• The first line of the commit message is not a full sentence, hence you can remove the ending period.
• Just a reminder that the "Approved by:" etc are tab-separated, e.g. Approved by:(tab) ygy (mentor)
• Again, there should be a space before "(mentor)", e.g. meta (mentor) instead of meta(mentor).