Re: [PR] fix: set dry_run to be optional. [airflow]

2023-11-25 Thread via GitHub
boring-cyborg[bot] commented on PR #35167: URL: https://github.com/apache/airflow/pull/35167#issuecomment-1826251307 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-11-25 Thread via GitHub
eladkal merged PR #35167: URL: https://github.com/apache/airflow/pull/35167 -- 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:

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-11-15 Thread via GitHub
Calder-Ty commented on PR #35167: URL: https://github.com/apache/airflow/pull/35167#issuecomment-1812520193 Any update on this PR? -- 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

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-10-30 Thread via GitHub
austinau commented on PR #35167: URL: https://github.com/apache/airflow/pull/35167#issuecomment-1785859421 Alright, I removed the whitespace, thanks! -- 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

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-10-27 Thread via GitHub
eladkal commented on code in PR #35167: URL: https://github.com/apache/airflow/pull/35167#discussion_r1374861524 ## tests/api_connexion/schemas/test_task_instance_schema.py: ## @@ -236,6 +236,22 @@ def test_success(self): } assert expected_result == result +

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-10-26 Thread via GitHub
utkarsharma2 commented on code in PR #35167: URL: https://github.com/apache/airflow/pull/35167#discussion_r1373794535 ## tests/api_connexion/schemas/test_task_instance_schema.py: ## @@ -236,6 +236,22 @@ def test_success(self): } assert expected_result ==

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-10-24 Thread via GitHub
Taragolis commented on code in PR #35167: URL: https://github.com/apache/airflow/pull/35167#discussion_r1370941702 ## tests/api_connexion/schemas/test_task_instance_schema.py: ## @@ -236,6 +236,22 @@ def test_success(self): } assert expected_result == result

Re: [PR] fix: set dry_run to be optional. [airflow]

2023-10-24 Thread via GitHub
boring-cyborg[bot] commented on PR #35167: URL: https://github.com/apache/airflow/pull/35167#issuecomment-1777934235 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution

[PR] fix: set dry_run to be optional. [airflow]

2023-10-24 Thread via GitHub
austinau opened a new pull request, #35167: URL: https://github.com/apache/airflow/pull/35167 Fixes #35166. Duplicates the solution in #34568. --- **^ Add meaningful description above** Read the **[Pull Request