Browse Source

ref: unify signature of validate_config (#74894)

<!-- Describe your PR here. -->
anthony sottile 7 months ago
parent
commit
17885521d9

+ 1 - 1
src/sentry_plugins/asana/plugin.py

@@ -177,7 +177,7 @@ class AsanaPlugin(CorePluginMixin, IssuePlugin2):
     def get_issue_url(self, group, issue_id, **kwargs):
         return "https://app.asana.com/0/0/%s" % issue_id
 
-    def validate_config(self, project, config, actor):
+    def validate_config(self, project, config, actor=None):
         """
         ```
         if config['foo'] and not config['bar']:

+ 1 - 1
src/sentry_plugins/phabricator/plugin.py

@@ -159,7 +159,7 @@ class PhabricatorPlugin(CorePluginMixin, IssuePlugin2):
             )
         ]
 
-    def validate_config(self, project, config, actor):
+    def validate_config(self, project, config, actor=None):
         projectPHIDs = config.get("projectPHIDs")
         if projectPHIDs:
             try:

+ 1 - 1
src/sentry_plugins/pushover/plugin.py

@@ -88,7 +88,7 @@ class PushoverPlugin(CorePluginMixin, NotifyPlugin):
             },
         ]
 
-    def validate_config(self, project, config, actor):
+    def validate_config(self, project, config, actor=None):
         if int(config["priority"]) == 2 and config["retry"] < 30:
             retry = str(config["retry"])
             self.logger.exception(str(f"Retry not 30 or higher. It is {retry}."))

+ 1 - 1
src/sentry_plugins/redmine/plugin.py

@@ -231,7 +231,7 @@ class RedminePlugin(CorePluginMixin, IssuePlugin):
 
         return self.fields
 
-    def validate_config(self, project, config, actor):
+    def validate_config(self, project, config, actor=None):
         super().validate_config(project, config, actor)
         self.client_errors = []