Confirmed users
1,198
edits
Phabricator/FAQ: Difference between revisions
Jump to navigation
Jump to search
no edit summary
No edit summary |
No edit summary |
||
| Line 87: | Line 87: | ||
Anyone using Phabricator is authoring patches, reviewing patches, and/or accessing confidential patches. We feel MFA is a reasonable burden for these people given the importance of maintaining secure processes in the development of Firefox and other Mozilla apps. Given that Bugzilla is coupled to Phabricator, and a lot of important discussion around bugs and solutions also happens in Bugzilla, including confidential discussions, anyone involved in the above activities on Phabricator should also have equal protections when accessing Bugzilla. | Anyone using Phabricator is authoring patches, reviewing patches, and/or accessing confidential patches. We feel MFA is a reasonable burden for these people given the importance of maintaining secure processes in the development of Firefox and other Mozilla apps. Given that Bugzilla is coupled to Phabricator, and a lot of important discussion around bugs and solutions also happens in Bugzilla, including confidential discussions, anyone involved in the above activities on Phabricator should also have equal protections when accessing Bugzilla. | ||
== | == 'moz-phab' and 'arc' == | ||
=== How do I get “arc diff” to stop listing and asking about all my untracked build artifacts? === | === How do I get “arc diff” to stop listing and asking about all my untracked build artifacts? === | ||
| Line 142: | Line 142: | ||
See [https://secure.phabricator.com/book/phabricator/article/arcanist_windows/ Arcanist User Guide: Windows] for instructions on how to set this correctly. | See [https://secure.phabricator.com/book/phabricator/article/arcanist_windows/ Arcanist User Guide: Windows] for instructions on how to set this correctly. | ||
=== What does the 'contains arc fields' error mean? === | |||
The "contains arc fields" error means that revision was previously pushed to Phabricator with arc directly. | |||
moz-phab doesn't support all the fields that arc itself does, including the "test plan" field. If moz-phab were to rewrite the commit description that information would be lost. Currently moz-phab takes a conservative approach and will stop with the "contains arc fields" error message. | |||
You can work around that by rewriting the commit description yourself to only include the description and the 'Differential Revision:' line. | |||
There are plans to both support the "test plan" field and parse an arc formatted commit description with moz-phab. | |||
== Lando == | == Lando == | ||