[meta] Use Phabricator for all mozilla-central reviews
Categories
(Conduit :: General, enhancement, P1)
Tracking
(Not tracked)
People
(Reporter: mcote, Unassigned)
References
Details
(Keywords: conduit-story, conduit-triaged, meta)
User Story
When we are ready to resolve this bug, we should first disable the review flag for the following products: Core Firefox Firefox Build System NSPR NSS Toolkit
This is a tracker bug for work that must be completed before we require that all code reviews for mozilla-central patches be performed in Phabricator.
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
Please fix either bug 1426557 or bug 1495394 before forcing me to install Phabricator dependencies.
Reporter | ||
Comment 2•5 years ago
|
||
Hi, thanks for the comment, but please don't modify the blockers directly. The current plan is to make moz-phab (our officially supported, commit-stack-friendly command-line tool) no longer require Arcanist. After that it will be integrated into mach in some way. However, that work may not be completed before we transition all mozilla-central reviews to Phabricator. I realize installing Arcanist can be annoying, but more than 80% of the commits currently going into mozilla-central are being reviewed in Phabricator, so it appears that installing Arcanist is not a major blocker to adoption.
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Please don't add new blockers to this bug; needinfo myself or kmoir on a bug to nominate it for blocking.
Comment 4•5 years ago
|
||
Why not including "GeckoView" and "Firefox for Android"? These products use m-c.
(In reply to Makoto Kato [:m_kato] from comment #4)
Why not including "GeckoView" and "Firefox for Android"? These products use m-c.
From the announcement:
The list includes the Core, Firefox, Firefox Build System, NSS, Geckoview
and Toolkit products. It will also include some components of the Release
Engineering and Testing products. It is entirely possible that we will miss
some products/components, but users should not treat such omissions as
invitations to continue to use Bugzilla for mozilla-central reviews.
Comment 6•5 years ago
|
||
Comment 7•5 years ago
|
||
Closing. Thanks to everyone for all your work on this!
Description
•