shahar1 opened a new pull request, #38518:
URL: https://github.com/apache/airflow/pull/38518
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pu
uranusjr commented on code in PR #38518:
URL: https://github.com/apache/airflow/pull/38518#discussion_r1540186628
##
tests/providers/sftp/triggers/test_sftp.py:
##
@@ -175,13 +175,10 @@ async def
test_sftp_trigger_run_trigger_failure_state(self, mock_get_files_by_pa
Taragolis commented on code in PR #38518:
URL: https://github.com/apache/airflow/pull/38518#discussion_r1540205185
##
tests/providers/sftp/triggers/test_sftp.py:
##
@@ -175,13 +175,10 @@ async def
test_sftp_trigger_run_trigger_failure_state(self, mock_get_files_by_pa
Taragolis commented on code in PR #38518:
URL: https://github.com/apache/airflow/pull/38518#discussion_r1540207045
##
tests/providers/sftp/hooks/test_sftp.py:
##
@@ -405,12 +405,11 @@ def test_deprecation_ftp_conn_id(self,
mock_get_connection):
@mock.patch("airflow.provide
eladkal commented on PR #38518:
URL: https://github.com/apache/airflow/pull/38518#issuecomment-2025568339
Can you rebase and resolve conflicts?
--
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
potiuk commented on PR #38518:
URL: https://github.com/apache/airflow/pull/38518#issuecomment-2032877859
Or we can merge #38525 instead - @eladkal ?
--
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 t
shahar1 commented on code in PR #38518:
URL: https://github.com/apache/airflow/pull/38518#discussion_r1550356533
##
tests/providers/sftp/triggers/test_sftp.py:
##
@@ -175,13 +175,10 @@ async def
test_sftp_trigger_run_trigger_failure_state(self, mock_get_files_by_pa
t
potiuk commented on PR #38518:
URL: https://github.com/apache/airflow/pull/38518#issuecomment-2035482917
🤦
--
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,
potiuk merged PR #38518:
URL: https://github.com/apache/airflow/pull/38518
--
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: commits-unsubscr...@airflow.a