Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/type error #709

Merged
merged 2 commits into from
Jul 25, 2022
Merged

Fix/type error #709

merged 2 commits into from
Jul 25, 2022

Conversation

dforsi
Copy link
Contributor

@dforsi dforsi commented Jul 16, 2022

This PR adds a %s and a %u to two formatting strings and relevant tests.

dforsi added 2 commits July 16, 2022 15:31
Found with:
pylint --enable too-many-format-args

Fixes:
runtime error
TypeError: not all arguments converted during string formatting

pylint error
E1305: Too many arguments for format string (too-many-format-args)
Found with:
pylint --enable too-many-format-args

Fixes:
runtime error
TypeError: not all arguments converted during string formatting

pylint error
E1305: Too many arguments for format string (too-many-format-args)
@peterbarker peterbarker merged commit 99a1fef into ArduPilot:master Jul 25, 2022
@peterbarker
Copy link
Contributor

Merged, thanks!

@dforsi dforsi deleted the fix/type-error branch July 30, 2022 17:54
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