ref: unify the signature of Interface.to_string

src/sentry/interfaces/base.py

@@ -155,7 +155,7 @@ def get_score(self) -> int:
     def iter_tags(self):
         return iter(())
 
-    def to_string(self, event, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         return ""
 
     def to_email_html(self, event, **kwargs):

src/sentry/interfaces/exception.py

@@ -454,7 +454,7 @@ def get_api_meta(self, meta, is_public=False, platform=None):
 
         return {"values": result}
 
-    def to_string(self, event, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         if not self.values:
             return ""
 
@@ -471,7 +471,7 @@ def to_string(self, event, is_public=False, **kwargs):
                     )
                     + "\n\n"
                 )
-        return ("".join(output)).strip()
+        return "".join(output).strip()
 
     def get_stacktrace(self, *args, **kwargs):
         exc = self.values[-1]

src/sentry/interfaces/message.py

@@ -47,5 +47,5 @@ def to_json(self):
             {"message": self.message, "formatted": self.formatted, "params": self.params or None}
         )
 
-    def to_string(self, event, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         return self.formatted or self.message

src/sentry/interfaces/security.py

@@ -172,7 +172,7 @@ def to_python(cls, data, **kwargs):
         data.setdefault("effective_directive", None)
         return super().to_python(data, **kwargs)
 
-    def to_string(self, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         return orjson.dumps(
             {"csp-report": self.get_api_context()},
             option=orjson.OPT_UTC_Z | orjson.OPT_NON_STR_KEYS,

src/sentry/interfaces/stacktrace.py

@@ -325,7 +325,7 @@ def is_unhashable_function(self):
         # queries and JSON data)
         return self.function.startswith(("lambda$", "[Anonymous"))
 
-    def to_string(self, event):
+    def to_string(self, event) -> str:
         if event.platform is not None:
             choices = [event.platform]
         else:
@@ -525,7 +525,7 @@ def to_json(self):
             }
         )
 
-    def to_string(self, event, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         return self.get_stacktrace(event, system_frames=False, max_frames=10)
 
     def get_stacktrace(

src/sentry/interfaces/template.py

@@ -45,7 +45,7 @@ def to_python(cls, data, **kwargs):
 
         return super().to_python(data, **kwargs)
 
-    def to_string(self, event, is_public=False, **kwargs):
+    def to_string(self, event) -> str:
         context = get_context(
             lineno=self.lineno,
             context_line=self.context_line,

tests/sentry_plugins/victorops/test_plugin.py

@@ -16,7 +16,7 @@
 
 
 class UnicodeTestInterface(Interface):
-    def to_string(self, event):
+    def to_string(self, event) -> str:
         return self.body
 
     def get_title(self):