Update "Sending patch" documentation
Categories
(DevTools :: Documentation, task)
Tracking
(firefox72 fixed)
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
Details
Attachments
(1 file)
Bug 1596342 - Update sending patch documentation to only mention Phabricator + moz-phab. r=rcaliman.
47 bytes,
text/x-phabricator-request
|
Details | Review |
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"
Assignee | ||
Comment 1•1 year ago
|
||
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.
Updated•1 year ago
|
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.
Comment 3•1 year ago
|
||
bugherder |
Description
•