Add a base class for CI configuration forms.
Review Request #13866 — Created May 19, 2024 and submitted
We had a couple of form fields that were common to all CI configuration
forms, and have another one coming shortly. To avoid continuing to
duplicate fields, and to ease the development of new CI integrations,
this change introduces a newBaseCIIntegrationConfigForm
class that
can house these fields.All CI integrations now use this form as a base. The Travis-CI
integration does need to override theconditions
field in order to
limit to GitHub repositories, but inherits everything else.
Unit tests pass.
Tested each CI configuration form.
Summary | ID |
---|---|
a35b7a5ad6f273a60c90ca923acb17beca7a6ec7 |
Description | From | Last Updated |
---|---|---|
'djblets.forms.widgets.AmountSelectorWidget' imported but unused Column: 1 Error code: F401 |
![]() |