Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ repos:
- tomli

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.14.14
rev: v0.15.0
hooks:
- id: ruff-check
args: [ --fix ]
Expand Down
1 change: 0 additions & 1 deletion ardupilot_methodic_configurator/mavftp_example.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ def get_file(mav_ftp: mavftp.MAVFTP, remote_filename: str, local_filename: str,
mav_ftp.cmd_get([remote_filename, local_filename])
ret = mav_ftp.process_ftp_reply("OpenFileRO", timeout=timeout)
ret.display_message()
# mav_ftp.session = session # FIXME: this is a huge workaround hack # pylint: disable=fixme
debug_class_member_variable_changes(mav_ftp)
# time.sleep(0.2)

Expand Down
54 changes: 32 additions & 22 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -178,14 +178,6 @@ exclude = [

# https://docs.astral.sh/ruff/rules/
lint.select = [
"F", # Pyflakes
"E", # pycodestyle -Error
"W", # pycodestyle - Warning
"C90", # maccabe
"I", # isort
"N", # pep8-naming
"D", # pydocstyle
"UP", # pyupgrade
#"ERA", # eradicate
"YTT", # flake8-2020
"ANN", # flake8-annotations
Expand All @@ -197,40 +189,56 @@ lint.select = [
"A", # flake8-builtins
"COM", # flake8-commas
"C4", # flake8-comprehensions
#"CPY", # flake8-copyright
"DTZ", # flake8-datetimez
"T10", # flake8-debugger
"EM", # flake8-errmsg
"EXE", # flake8-executable
#"FIX", # flake8-fixme
"FIX", # flake8-fixme
"FA", # flake8-future-annotations
"INT", # flake8-gettext
"ISC", # flake8-implicit-str-concat
"ICN", # flake8-import-conventions
"LOG", # flake8-logging
"G", # flake8-logging-format
"INP", # flake8-no-pep420
"PIE", # flake8-pie
"T20", # flake8-print (T20)
"PYI", # flake8-pyi (PYI)
"PT", # flake8-pytest-style (PT)
"Q", # flake8-quotes (Q)
"RSE", # flake8-raise (RSE)
"RET", # flake8-return (RET)
"SLF", # flake8-self (SLF)
"T20", # flake8-print
"PYI", # flake8-pyi
"PT", # flake8-pytest-style
"Q", # flake8-quotes
"RSE", # flake8-raise
"RET", # flake8-return
"SLF", # flake8-self
"SIM", # flake8-simplify
"SLOT", # flake8-slots (SLOT)
"SLOT", # flake8-slots
"TID", # flake8-tidy-imports
#"TD", # flake8-todos
"TD", # flake8-todos
"TC", # flake8-type-checking
"ARG", # flake8-unused-arguments
#"PTH", # flake8-use-pathlib
"INT", # flake8-gettext
"FLY", # flynt
"I", # isort
"C90", # maccabe
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in the comment: this linter is named “mccabe” (double ‘c’), not “maccabe”.

Suggested change
"C90", # maccabe
"C90", # mccabe

Copilot uses AI. Check for mistakes.
"NPY", # NumPy-specific rules
"PD", # pandas-vet
"N", # pep8-naming
"PERF", # Performance-related issues
"E", # pycodestyle -Error
"W", # pycodestyle - Warning
#"DOC", # pydoclint
"D", # pydocstyle
"F", # Pyflakes
"PGH", # pygrep-hooks
"PL", # Pylint
"FLY", # flynt
"PERF", # Performance-related issues
"PLC", # Convention
"PLE", # Error
"PLR", # Refactor
"PLW", # Warning
"UP", # pyupgrade
"FURB", # refurb
"DOC", # pydoclint
"RUF", # Ruff-specific codes
#"TRY", # tryceratops
]

lint.ignore = [
Expand All @@ -248,6 +256,8 @@ lint.ignore = [
"FBT001", # Boolean-typed positional argument in function definition
"FBT002", # Boolean default positional argument in function definition
"LOG015", # `error()` call on root logger, Use own logger instead
"FA100", # Add `from __future__ import annotations` to simplify xxx
"TRY400", # Use logging.exception instead of logging.error
Comment on lines +259 to +260
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TRY400 is listed in lint.ignore, but the TRY (tryceratops) ruleset isn’t enabled in lint.select (it’s commented out). This ignore entry is currently a no-op and can be confusing; either enable TRY or remove TRY400 from the ignore list until it’s needed.

Copilot uses AI. Check for mistakes.
]

line-length = 127
Expand Down
Loading