From 89bf76553e79ddf8746eaa3c8ce4b28b0a2ef2de Mon Sep 17 00:00:00 2001 From: Kasia Kucharczyk <2536609+kkucharc@users.noreply.github.com> Date: Tue, 29 Sep 2020 17:10:46 +0200 Subject: [PATCH] Updated lint in schedules.py in tasks: unused_import, ungrouped_import and changed argument name to remove unused-argument (#11045) --- superset/tasks/schedules.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/superset/tasks/schedules.py b/superset/tasks/schedules.py index 2cc1280f01085..39394c072f846 100644 --- a/superset/tasks/schedules.py +++ b/superset/tasks/schedules.py @@ -34,7 +34,7 @@ TYPE_CHECKING, Union, ) -from urllib.error import URLError # pylint: disable=ungrouped-imports +from urllib.error import URLError import croniter import simplejson as json @@ -72,7 +72,6 @@ # pylint: disable=too-few-public-methods if TYPE_CHECKING: - # pylint: disable=unused-import from flask_appbuilder.security.sqla.models import User from werkzeug.datastructures import TypeConversionDict @@ -485,8 +484,8 @@ def deliver_slice( # pylint: disable=too-many-arguments bind=True, soft_time_limit=config["EMAIL_ASYNC_TIME_LIMIT_SEC"], ) -def schedule_email_report( # pylint: disable=unused-argument - task: Task, +def schedule_email_report( + _task: Task, report_type: ScheduleType, schedule_id: int, recipients: Optional[str] = None, @@ -541,8 +540,8 @@ def schedule_email_report( # pylint: disable=unused-argument retry_kwargs={"max_retries": 5}, retry_backoff=True, ) -def schedule_alert_query( # pylint: disable=unused-argument - task: Task, +def schedule_alert_query( + _task: Task, report_type: ScheduleType, schedule_id: int, recipients: Optional[str] = None,