Skip to content

Commit

Permalink
Various ruff rule compliance (W, CPY, ISC, AIR) (SciTools#5667)
Browse files Browse the repository at this point in the history
* fix pycodestyle W

* fix for ruff: flake8-copyright (CPY)

* ruff, no fix needed: flake8-implicit-str-concat (ISC)

* ruff, no fix needed: Airflow (AIR)
  • Loading branch information
tkknight authored Jan 2, 2024
1 parent 1b61529 commit 41f1107
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 15 deletions.
13 changes: 0 additions & 13 deletions .ruff.toml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ lint.ignore = [
# pycodestyle (E, W)
# https://docs.astral.sh/ruff/rules/#pycodestyle-e-w
"E",
"W",

# mccabe (C90)
# https://docs.astral.sh/ruff/rules/#mccabe-c90
Expand Down Expand Up @@ -71,10 +70,6 @@ lint.ignore = [
# https://docs.astral.sh/ruff/rules/#flake8-builtins-a
"A",

# flake8-copyright (CPY)
# https://docs.astral.sh/ruff/rules/#flake8-copyright-cpy
"CPY",

# flake8-comprehensions (C4)
# https://docs.astral.sh/ruff/rules/#flake8-comprehensions-c4
"C4",
Expand All @@ -91,10 +86,6 @@ lint.ignore = [
# https://docs.astral.sh/ruff/rules/#flake8-future-annotations-fa
"FA",

# flake8-implicit-str-concat (ISC)
# https://docs.astral.sh/ruff/rules/#flake8-implicit-str-concat-isc
"ISC",

# flake8-logging-format (G)
# https://docs.astral.sh/ruff/rules/#flake8-logging-format-g
"G",
Expand Down Expand Up @@ -187,10 +178,6 @@ lint.ignore = [
# https://docs.astral.sh/ruff/rules/#flynt-fly
"FLY",

# Airflow (AIR)
# https://docs.astral.sh/ruff/rules/#airflow-air
"AIR",

# Perflint (PERF)
# https://docs.astral.sh/ruff/rules/#perflint-perf
"PERF",
Expand Down
4 changes: 2 additions & 2 deletions benchmarks/bm_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,8 +257,8 @@ def _gh_create_reports(commit_sha: str, results_full: str, results_shifts: str)
* commit {commit_sha} ({pr_tag}).
<p>
Please review the report below and
take corrective/congratulatory action as appropriate
Please review the report below and
take corrective/congratulatory action as appropriate
:slightly_smiling_face:
</p>
"""
Expand Down

0 comments on commit 41f1107

Please sign in to comment.