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

Add comments to metaclasses. #635

Merged
merged 6 commits into from
Oct 7, 2024

Conversation

junkmd
Copy link
Collaborator

@junkmd junkmd commented Oct 7, 2024

During the investigation for #618, I noticed that one metaclass had comments while the other didn't, so I added the missing comments.

Additionally, I found some implementations that seem to be remnants from the time when Python 2.x was supported, so I added # HACK comments.
I plan to open issues regarding these matters soon.

@junkmd junkmd added this to the 1.4.8 milestone Oct 7, 2024
junkmd added a commit to junkmd/pywinauto that referenced this pull request Oct 7, 2024
to `_coclass_meta.__new__`.
to `_cominterface_meta.__new__`.
@junkmd junkmd force-pushed the add_comments_to_metaclasses branch from b1b9810 to 4900009 Compare October 7, 2024 15:49
junkmd added a commit to junkmd/pywinauto that referenced this pull request Oct 7, 2024
@junkmd junkmd merged commit 07e0664 into enthought:main Oct 7, 2024
41 checks passed
@junkmd junkmd deleted the add_comments_to_metaclasses branch October 7, 2024 23:29
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