CR changes

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 ## v0.4.51
 **New Features**:
-- Sinks: From now on, no need to explicitly publish playbook results to a specific target. 
+- Sinks - **Breaking Changes**: From now on, no need to explicitly publish playbook results to a specific target. 
 
   Within the playbook's code we only need to create a ``Finding`` object. It will be published to the configured ``sinks``
 

docs/user-guide/playbook-configuration.rst

@@ -94,7 +94,7 @@ Playbooks sinks
 
 
 | **Note:** Create your Robusta account, to get the ``token`` for the Robusta sink `here <https://robusta.dev>`_ .
-| **Note:** Slack sink ``api_key`` is added during the installation process.
+| **Note:** The Slack sink ``api_key`` is added during the installation process.
 | In order to get a new key run: ``robusta integrations get-integration-key --integration-type slack``.
 
 | By default, all playbooks will forward the results to the default sinks.

src/robusta/integrations/slack/sender.py

@@ -63,6 +63,7 @@ def __apply_length_limit(msg: str, max_length: int = 3000):
         truncator = "..."
         return msg[: max_length - len(truncator)] + truncator
 
+    @staticmethod
     def __to_slack_diff(block: DiffsBlock, sink_name: str) -> List[SlackBlock]:
         num_additions = len([d for d in block.diffs if d.diff_type == DiffType.ADDED])
         num_subtractions = len(
@@ -92,6 +93,7 @@ def __to_slack_diff(block: DiffsBlock, sink_name: str) -> List[SlackBlock]:
 
         return slack_blocks
 
+    @staticmethod
     def __to_slack(block: BaseBlock, sink_name: str) -> List[SlackBlock]:
         if isinstance(block, MarkdownBlock):
             if not block.text: