1522067 - Add additional info about commit messages & blocking reviews

Status: RESOLVED FIXED

(DevTools :: General, enhancement, P3)

Description

[Ola Gasidlo [:ogasidlo] [:ola] OOO - please ping :ladybenko:]

devtools/docs/contributing/making-prs.md - Line 7

Change:

---

First, commit your changes. For example:

hg add /path/to/file/changed
hg commit -m "Bug 1234567 - Implement feature XYZ."

---

to

---

First, commit your changes. For example:

hg add /path/to/file/changed
hg commit -m "Bug 1234567 - Implement feature XYZ. r=name,name2!"

The commit message explained in detail:

• Bug 1234567 - The number of the bug in bugzilla.
• - Implement feature XYZ. - The commit message.
• r=name - The short form to request a review.
• ,name2! - You can have more than one reviewer. The ! makes the review a blocking review (Patch can not land without accepted review). You will also need to add it to the name in the popup after you ran arc diff.

Please note your first commit message will be also the title of your patch in Phabricator. (Don't worry, if something goes wrong. You can still change things in the UI later).

---

Comment 1

[Manish [:manishkk]]

I think, r?name will be more suitable for commits instead of r=name

to

---

First, commit your changes. For example:

hg add /path/to/file/changed
hg commit -m "Bug 1234567 - Implement feature XYZ. r=name, name2!"

Comment 2

[Ola Gasidlo [:ogasidlo] [:ola] OOO - please ping :ladybenko:]

Phabricator is able to parse both r?name as well as r=name.

The recommendation for devtools is r=name.

Comment 3

[Shivam Singhal [ :championshuttler ]]

Attachment: Bug 1522067 - Add additional info about commit messages & blocking reviews. r=ola

Add additional info about commit messages and blocking reviews.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a1f671e9526
Add additional info about commit messages & blocking reviews. r=Ola

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/3a1f671e9526