Skip to content

Commit

Permalink
fix: Fix new pylint warnings.
Browse files Browse the repository at this point in the history
* `no-self-use` is now an optional value. We don't need to disable it
  manually because it's disabled by default.

* We had some superflous parens in the symmath_check.py file so we
  remove those as well.
  • Loading branch information
feanil committed Mar 20, 2024
1 parent b23a8c9 commit 5a6acb1
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,6 @@ disable =
illegal-waffle-usage,

logging-fstring-interpolation,
no-self-use,
django-not-available,

[REPORTS]
Expand Down Expand Up @@ -385,4 +384,4 @@ int-import-graph =
[EXCEPTIONS]
overgeneral-exceptions = builtins.Exception

# 37fb074922e492cc9d7946220ba214f767ee70f1
# f90fb567b1f44429c280e9de9c5d874fb1d81d25
1 change: 0 additions & 1 deletion pylintrc_tweaks
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,4 @@ load-plugins = edx_lint.pylint,pylint_django,pylint_celery

[MESSAGES CONTROL]
disable+ =
no-self-use,
django-not-available,
2 changes: 1 addition & 1 deletion symmath/symmath_check.py
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ def symmath_check(expect, ans, dynamath=None, options=None, debug=None, xml=None
#fsym = fsym.simplify()
#fexpect = fexpect.simplify()
try:
diff = (fexpect - fsym)
diff = fexpect - fsym
except Exception as err: # lint-amnesty, pylint: disable=broad-except
diff = None

Expand Down

0 comments on commit 5a6acb1

Please sign in to comment.