gopidesupavan commented on code in PR #54061:
URL: https://github.com/apache/airflow/pull/54061#discussion_r2250617975
##########
providers/slack/src/airflow/providers/slack/hooks/slack.py:
##########
@@ -237,7 +237,6 @@ def send_file_v1_to_v2(
initial_comment: str | None = None,
title: str | None = None,
snippet_type: str | None = None,
- **kwargs,
Review Comment:
Yes it doesnt serve any functionality, as we are not at all referring kwrgs
in that function. if users sending any extract function args it will fail now,
earlier with kwargs its accepting any kind args though we are not using.
If you think we should keep that i am happy to keep, it just that i dont
want keep that as we are not using at all in that function and its more
confusion why kwargs used for ;)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
