Skip to content

AP_Generator: remove Python3 suppor from test-loweheiser.py #30010

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

Merged
merged 1 commit into from
May 14, 2025

Conversation

peterbarker
Copy link
Contributor

No description provided.

@amilcarlucas
Copy link
Contributor

Correct the commit message:

AP_Generator: remove Python2 support from test-loweheiser.py

@peterbarker peterbarker force-pushed the pr/remove-pruntfunc branch from 7daba23 to c579ea3 Compare May 9, 2025 02:34
@peterbarker
Copy link
Contributor Author

Correct the commit message:

AP_Generator: remove Python2 support from test-loweheiser.py

Done, thanks.

@tpwrules
Copy link
Contributor

Doesn’t look corrected to me.

@peterbarker
Copy link
Contributor Author

Doesn’t look corrected to me.

Are you mixing up the PR title with the commit message?
image

@tpwrules
Copy link
Contributor

They both say Python 3?

@peterbarker peterbarker force-pushed the pr/remove-pruntfunc branch from c579ea3 to 971fe5b Compare May 13, 2025 03:28
@peterbarker
Copy link
Contributor Author

They both say Python 3?

Quite right, should have had more words in this. It's more compatability code or whatever.

More words added.

In place while we were supporting both Py2 and Py3, no longer required
@peterbarker peterbarker force-pushed the pr/remove-pruntfunc branch from 971fe5b to a35eb12 Compare May 13, 2025 12:08
@peterbarker peterbarker merged commit d4e5013 into ArduPilot:master May 14, 2025
123 of 124 checks passed
@peterbarker peterbarker deleted the pr/remove-pruntfunc branch May 14, 2025 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants