-
Notifications
You must be signed in to change notification settings - Fork 3k
Updates to tools for Python 3 compatibility #11921
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
Conversation
@madchutney, thank you for your changes. |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
This would be good to have reviewed within today to have in 5.15, please review |
@mark-edgeworth @acabarbaye please review |
I spoke to @mark-edgeworth , the update will come tomorrow (internal review/tests found some issues) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Over-the-shoulder review done with Graham on Monday. All now fixed.
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Summary of change
Final changes to support Python 3 in the
/tools
directory owned by B-Tools.Documentation
Remove any mention of the tools supporting Python 2 only.
Pull request type
Test results
Reviewers
@mark-edgeworth
@jamesbeyond
Release Notes
Python 3 is now supported by the tools.
Summary of changes
No functional changes, code should be equivalent.
Impact of changes
Code should be Python 3 compatible
Migration actions required
None