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 authored and auxten committed Nov 20, 2020
1 parent a589e25 commit 067a092
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 067a092

Please sign in to comment.