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

Remove Python2-related __future__ imports #1640

Conversation

vergeev
Copy link
Contributor

@vergeev vergeev commented Feb 13, 2024

Closes #1636

Removes every from __future__ import print_function outside of the vendor directory.

Also removes other Python2-related imports, such as:

  • from __future__ import with_statement
  • from __future__ import absolute_import

@vergeev vergeev requested a review from a team as a code owner February 13, 2024 18:27
Copy link

linux-foundation-easycla bot commented Feb 13, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Signed-off-by: Pavel Vergeev <vernier_06_curios@icloud.com>
@vergeev vergeev force-pushed the chore/remove-import-futures branch from 4b6360e to cd5dbff Compare February 13, 2024 18:27
Signed-off-by: Pavel Vergeev <vernier_06_curios@icloud.com>
@JeanChristopheMorinPerso
Copy link
Member

Thank you @vergeev! I triggered builds to see if everything passes.

Can I ask you if you found this issue through https://clotributor.dev/search?foundation=aswf?

@JeanChristopheMorinPerso JeanChristopheMorinPerso merged commit 6053450 into AcademySoftwareFoundation:main Feb 13, 2024
52 checks passed
@vergeev
Copy link
Contributor Author

vergeev commented Feb 14, 2024

Can I ask you if you found this issue through https://clotributor.dev/search?foundation=aswf?

I did find it through CLOTributor, yes :)

@vergeev vergeev deleted the chore/remove-import-futures branch February 14, 2024 06:51
Pixel-Minions pushed a commit to Pixel-Minions/rez that referenced this pull request Feb 14, 2024
…1640)

Signed-off-by: Pavel Vergeev <vernier_06_curios@icloud.com>
Signed-off-by: Jose Enriquez <jose.enriquez@barnstormvfx.com>
Pixel-Minions added a commit to Pixel-Minions/rez that referenced this pull request Sep 26, 2024
…1640)

Signed-off-by: Pavel Vergeev <vernier_06_curios@icloud.com>
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.

Remove from __future__ import print_function from the code base
2 participants