[airflow] 08/14: Update SendGridEmail with None return type (#11256)

[po...@apache.org]

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 <32...@users.noreply.github.com>
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