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 a python 3.12 compatibility issue #7450

Merged
merged 1 commit into from
May 24, 2024

Conversation

rvianello
Copy link
Contributor

What does this implement/fix? Explain your changes.

This PR fixes a small regression in the compatibility with python 3.12

Copy link
Member

@greglandrum greglandrum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@greglandrum greglandrum added the Cleanup Code cleanup and refactoring label May 24, 2024
@greglandrum greglandrum added this to the 2024_03_3 milestone May 24, 2024
@greglandrum greglandrum added the not in release notes a change that doesn't need to go in the release notes label May 24, 2024
@greglandrum greglandrum merged commit b5041c3 into rdkit:master May 24, 2024
12 checks passed
@greglandrum
Copy link
Member

Thanks for cleaning up after me @rvianello. I will switch my build envs to python 3.12 in order to try and make sure this doesn't happen again

@rvianello
Copy link
Contributor Author

thank you @greglandrum and no problem, the issue was really trivial.

syedzayyan pushed a commit to syedzayyan/rdkit that referenced this pull request Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cleanup Code cleanup and refactoring not in release notes a change that doesn't need to go in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants