Change integration base classes to be more clearly defined as base classes
This issue came out of this discussion !61744 (comment 576987150).
About
We have some integration models that are used as base classes, but are not named as such, so they appear as though they are actual integrations.
We want to rename these models, and also (edit: done) mark them as abstract classes with self.abstract_class = true
.
The integration base classes are:
Integrations::BaseIssueTracker
Integrations::BaseMonitoring
Integrations::BaseSlashCommands
Integrations::BaseChatNotification
Integrations::BaseCi
Proposal
Rename the classes
Rename these classes to:
-
renamed in !63019 (merged)Integrations::BaseIssueTracker
-
renamed in !62645 (merged)Integrations::BaseMonitoring
-
renamed in !63015 (merged)Integrations::BaseSlashCommands
-
renamed in !61957 (merged)Integrations::BaseChatNotification
-
renamed in !62359 (merged)Integrations::BaseCi
Mark them all as abstract classes
Add self.abstract_class = true
to these classes. won't do #331181 (comment 1042022271).
Availability & Testing
- run
package-and-qa
- consider overriding
base_class
on integrations that inherit from the base classes to match existing behavior as per this comment: #331181 (comment 1034209944)
Edited by Arturo Herrero