changes from python setup.py validate

Author: ruberVulpes

slack/web/async_client.py

@@ -123,29 +123,60 @@ async def admin_apps_restricted_list(self, **kwargs) -> AsyncSlackResponse:
         )
 
     async def admin_conversations_restrictAccess_addGroup(
-        self, **kwargs
+        self, *, channel_id: str, group_id: str, **kwargs
     ) -> AsyncSlackResponse:
-        """Add an allowlist of IDP groups for accessing a channel."""
+        """Add an allowlist of IDP groups for accessing a channel.
+
+        Args:
+            channel_id (str): The channel to link this group to. e.g. 'C1234567890'
+            group_id (str): The IDP Group ID to be an allowlist for the private channel. 'S0604QSJC'
+            team_id (str): The workspace where the channel exists.
+                This argument is required for channels only tied to one workspace,
+                and optional for channels that are shared across an organization.
+                e.g 'T1234'
+        """
+        kwargs.update({"channel_id": channel_id, "group_id": group_id})
         return await self.api_call(
             "admin.conversations.restrictAccess.addGroup",
             http_verb="GET",
             params=kwargs,
         )
 
     async def admin_conversations_restrictAccess_listGroups(
-        self, **kwargs
+        self, *, channel_id: str, **kwargs
     ) -> AsyncSlackResponse:
-        """List all IDP Groups linked to a channel."""
+        """List all IDP Groups linked to a channel.
+
+        Args:
+            channel_id (str): The channel to link this group to. e.g. 'C1234567890'
+            team_id (str): The workspace where the channel exists.
+                This argument is required for channels only tied to one workspace,
+                and optional for channels that are shared across an organization.
+                e.g 'T1234'
+        """
+        kwargs.update({"channel_id": channel_id})
         return await self.api_call(
             "admin.conversations.restrictAccess.listGroups",
             http_verb="GET",
             params=kwargs,
         )
 
     async def admin_conversations_restrictAccess_removeGroup(
-        self, **kwargs
+        self, *, channel_id: str, group_id: str, team_id: str, **kwargs
     ) -> AsyncSlackResponse:
-        """Remove a linked IDP group linked from a private channel."""
+        """Remove a linked IDP group linked from a private channel.
+
+        Args:
+            channel_id (str): The channel to link this group to. e.g. 'C1234567890'
+            group_id (str): The IDP Group ID to be an allowlist for the private channel. 'S0604QSJC'
+            team_id (str): The workspace where the channel exists.
+                This argument is required for channels only tied to one workspace,
+                and optional for channels that are shared across an organization.
+                e.g 'T1234'
+        """
+        kwargs.update(
+            {"channel_id": channel_id, "group_id": group_id, "team_id": team_id}
+        )
         return await self.api_call(
             "admin.conversations.restrictAccess.removeGroup",
             http_verb="GET",

slack/web/client.py

@@ -111,11 +111,7 @@ def admin_apps_restricted_list(self, **kwargs) -> Union[Future, SlackResponse]:
         )
 
     def admin_conversations_restrictAccess_addGroup(
-        self,
-        *,
-        channel_id: str,
-        group_id: str,
-        **kwargs
+        self, *, channel_id: str, group_id: str, **kwargs
     ) -> Union[Future, SlackResponse]:
         """Add an allowlist of IDP groups for accessing a channel.
 
@@ -127,18 +123,15 @@ def admin_conversations_restrictAccess_addGroup(
                 and optional for channels that are shared across an organization.
                 e.g 'T1234'
         """
-        kwargs.update({"channel_id": channel_id, 'group_id': group_id})
+        kwargs.update({"channel_id": channel_id, "group_id": group_id})
         return self.api_call(
             "admin.conversations.restrictAccess.addGroup",
             http_verb="GET",
             params=kwargs,
         )
 
     def admin_conversations_restrictAccess_listGroups(
-        self,
-        *,
-        channel_id: str,
-        **kwargs
+        self, *, channel_id: str, **kwargs
     ) -> Union[Future, SlackResponse]:
         """List all IDP Groups linked to a channel.
 
@@ -157,12 +150,7 @@ def admin_conversations_restrictAccess_listGroups(
         )
 
     def admin_conversations_restrictAccess_removeGroup(
-        self,
-        *,
-        channel_id: str,
-        group_id: str,
-        team_id: str,
-        **kwargs
+        self, *, channel_id: str, group_id: str, team_id: str, **kwargs
     ) -> Union[Future, SlackResponse]:
         """Remove a linked IDP group linked from a private channel.
 
@@ -174,7 +162,9 @@ def admin_conversations_restrictAccess_removeGroup(
                 and optional for channels that are shared across an organization.
                 e.g 'T1234'
         """
-        kwargs.update({"channel_id": channel_id, 'group_id': group_id, 'team_id': team_id})
+        kwargs.update(
+            {"channel_id": channel_id, "group_id": group_id, "team_id": team_id}
+        )
         return self.api_call(
             "admin.conversations.restrictAccess.removeGroup",
             http_verb="GET",