Skip to content

Commit

Permalink
Updated lint in schedules.py in tasks: unused_import, ungrouped_impor…
Browse files Browse the repository at this point in the history
…t and changed argument name to remove unused-argument (apache#11045)
  • Loading branch information
kkucharc committed Sep 29, 2020
1 parent 5d08a42 commit 89bf765
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions superset/tasks/schedules.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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

Expand Down Expand Up @@ -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,
Expand Down Expand Up @@ -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,
Expand Down

0 comments on commit 89bf765

Please sign in to comment.