Closed Bug 1596342 Opened 5 months ago Closed 5 months ago

Update "Sending patch" documentation

Categories

(DevTools :: Documentation, task)

task
Not set

Tracking

(firefox72 fixed)

RESOLVED FIXED
Firefox 72
Tracking Status
firefox72 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

Details

Attachments

(1 file)

The documentation mention both arc and moz-phab which is confusing for user.
Since moz-phab don't rely on arc anymore and is the tool mozilla is investing in, I think we should remove any mention of arc.

The documentation also mention attaching patches to Bugzilla, where we can't do review anymore. Even if it say it not recommended to do so, it can be overwhelming for the new user. I think we should only mention one workflow, which is the one that is "mozilla optimized"

The documentation was mentioning arc as the primary
tool to push to review (and was discouraging using
moz-phab). moz-phab is now much more stable and
is the prefered way of pushing patches, so we shouldn't
mention arc anymore.
There was also a (not recommended) mention of attaching
patch to Bugzilla for reviews, which I think isn't
something we recommend anymore.

Attachment #9108630 - Attachment description: Bug 1596342 - Update sending patch documentation to only mention Phabricator + moz-phab. r=pbro. → Bug 1596342 - Update sending patch documentation to only mention Phabricator + moz-phab. r=rcaliman.
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ee728f3ea085
Update sending patch documentation to only mention Phabricator + moz-phab. r=rcaliman.
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72
You need to log in before you can comment on or make changes to this bug.