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: Update codebase for Python 3 compatibility #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

am-niz
Copy link

@am-niz am-niz commented Mar 7, 2024

Description

What has been changed?

This pull request updates the codebase to be fully compatible with Python 3, addressing syntax changes and other incompatibilities found while running the project in a Python 3 environment.

Key changes include:

  • Updating print statements to use function syntax.
  • Modifying any relevant Python 2-specific library calls to their Python 3 equivalents.
  • Ensuring all tests pass under Python 3.

Why this change is necessary?

The project was not fully compatible with Python 3, which is the current standard for Python development. Making these updates ensures broader accessibility and usability of the project, keeping it up-to-date with ongoing Python development practices.

Additional context

  • All changes were tested locally to ensure functionality remains intact.
  • Wherever possible, care was taken to maintain backward compatibility with Python 2, though the primary focus was on Python 3 compatibility.

How has this been tested?

  • Describe the testing procedure, e.g., "Ran the existing test suite and all tests pass under Python 3."
  • Mention any additional testing or checks you performed, such as manual testing of specific functionality affected by the changes.

Screenshots (if appropriate)

(Include any relevant screenshots that help visualize the changes made.)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

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.

1 participant