Skip to content

fix pylint disable - #2013

Merged
amilcarlucas merged 1 commit into
masterfrom
fix_pylint
Sep 2, 2026
Merged

fix pylint disable#2013
amilcarlucas merged 1 commit into
masterfrom
fix_pylint

Conversation

@amilcarlucas

Copy link
Copy Markdown
Collaborator

Description

fix pylint disable

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

Copilot AI lite review requested due to automatic review settings September 2, 2026 09:50
@amilcarlucas
amilcarlucas merged commit ea20179 into master Sep 2, 2026
8 of 26 checks passed
@amilcarlucas
amilcarlucas deleted the fix_pylint branch September 2, 2026 09:51

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.

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Pull request overview

Moves the pylint: disable=too-many-locals directive to the correct location so it applies to the entire function definition.

Changes:

  • Relocated the pylint disable comment from the return type line to the def line for create_new_vehicle_from_bin_log.

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

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