Skip to content

chore(pylint): pylint fixes - #2234

Merged
amilcarlucas merged 1 commit into
masterfrom
lint
Oct 16, 2026
Merged

amilcarlucas merged 1 commit into
masterfrom
lint

Conversation

@amilcarlucas

Copy link
Copy Markdown
Collaborator

Description

pylint fixes

Checklist

  • Run pre-commit checks locally
  • Verified by a human programmer
  • All commits are signed off (use git commit --signoff)
  • Code follows our coding standards
  • Documentation updated if needed
  • No breaking changes or properly documented

Testing

Describe how you tested these changes:

  • Unit tests pass
  • Integration tests pass
  • Manual testing performed
  • Tested on flight controller hardware

Signed-off-by: Dr.-Ing. Amilcar do Carmo Lucas <amilcar.lucas@iav.de>
Copilot AI balanced review requested due to automatic review settings October 16, 2026 09:42
@amilcarlucas
amilcarlucas merged commit 19fc6a1 into master Oct 16, 2026
21 of 24 checks passed
@amilcarlucas
amilcarlucas deleted the lint branch October 16, 2026 09:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Both changes are limited to lint comments, with no identified blocking issues.

0 open findings

What changed in this PR

Adds targeted Pylint suppressions to two maintenance scripts without changing runtime behavior.

Changes:

  • Suppresses too-many-locals in template processing.
  • Suppresses too-many-nested-blocks in configuration-step string extraction.
File Description
update_vehicle_templates.py Adds a local-variable-count lint suppression.
update_configuration_steps_translation.py Adds a nesting-depth lint suppression.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown
Contributor

☂️ Code Coverage

current status: ✅

Overall Coverage

Statements Covered Coverage Threshold Status
26982 25090 93% 89% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 927de7d by action🐍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants