This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch v1-10-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 217387596c100888205f25df047aa14cabfc8855 Author: Patrick Cando <[email protected]> AuthorDate: Sat Oct 3 15:06:33 2020 +0100 Update SendGridEmail with None return type (#11256) --- airflow/upgrade/rules/send_grid_moved.py | 2 -- tests/upgrade/rules/test_send_grid_moved.py | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/airflow/upgrade/rules/send_grid_moved.py b/airflow/upgrade/rules/send_grid_moved.py index 7e3ddcd..0bab17e 100644 --- a/airflow/upgrade/rules/send_grid_moved.py +++ b/airflow/upgrade/rules/send_grid_moved.py @@ -36,5 +36,3 @@ The SendGrid module `airflow.contrib.utils.sendgrid` was moved to `airflow.provi msg = "Email backend option uses airflow.contrib Sendgrid module. " \ + "Please use new module: {}".format(email_provider_path) return [msg] - else: - return [] diff --git a/tests/upgrade/rules/test_send_grid_moved.py b/tests/upgrade/rules/test_send_grid_moved.py index 1a1bb45..807d186 100644 --- a/tests/upgrade/rules/test_send_grid_moved.py +++ b/tests/upgrade/rules/test_send_grid_moved.py @@ -42,6 +42,6 @@ class TestSendGridEmailerMovedRule(TestCase): assert isinstance(rule.description, str) assert isinstance(rule.title, str) - msg = [] + msg = None response = rule.check() assert response == msg
