This is an automated email from the ASF dual-hosted git repository.

turbaszek pushed a commit to branch v1-10-test
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/v1-10-test by this push:
     new 2b0dd5d  Update SendGridEmail with None return type (#11256)
2b0dd5d is described below

commit 2b0dd5dc73f4f50dce78eec8cd6d597b8c84341d
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

Reply via email to