It's possible to bypass warnings for no reviewers, failed lint, failed tests, etc., but it is not possible to bypass the error for an empty test plan. It should be possible to bypass this error as well, because it should be possible to submit a commit no matter how broken arc believes it is. Submitting broken commits is a common workflow for us to submit e.g. commits with deliberately failing tests to illustrate something that's broken, or submit commits that are nowhere near done as a work in progress / proof of concept.
Description
Description
Details
Details
- Reference
- fl312
| Status | Subtype | Assigned | Task | |
|---|---|---|---|---|
| · · · | ||||
| Declined | None | T167 Align basics of current Gerrit code-review process with Differential | ||
| Resolved | Qgil | T185 Make it possible to bypass missing test plan | ||
| · · · |
Event Timeline
Comment Actions
epriestley wrote on 2014-05-11 00:01:26 (UTC)
You can disable this field in differential.fields or make it optional with differential.require-test-plan-field in the configuration.
Comment Actions
qgil wrote on 2014-05-11 09:32:14 (UTC)
Thank you @epriestley. I have edited /config/group/differential/ in this instance.
differential.require-test-plan-field Customized Require "Test Plan" field? Current Value: false
Marking this task resolved, although we need to document this to apply this configuration when working on the actual Phabricator instance.