-
-
Notifications
You must be signed in to change notification settings - Fork 96
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
Deleted __cmp__() functions from repository #513
Conversation
Hi, Why was this closed? |
Hi Jun,
Sorry I had a project for a class where I had to complete a task for an open-source project. I didn't want to mess up anything with the project and I didn't get a chance to test out the software to make sure nothing was affected by removing the functions so I figured I would keep it on my own fork. I had to create a pull request to my own fork but accidentally created one to the main comptypes repo so that's why I closed it. If you would like me to, I can open it back up! I did two of the "good first tasks" where it was asked to delete some outdated functions
…-Alexander Calvo
________________________________
From: Jun Komoda ***@***.***>
Sent: Sunday, February 11, 2024 9:34 PM
To: enthought/comtypes ***@***.***>
Cc: Calvo, Alexander J. ***@***.***>; State change ***@***.***>
Subject: Re: [enthought/comtypes] Deleted __cmp__() functions from repository (PR #513)
[External Email]
Hi,
Why was this closed?
I think this pull request is helpful to the project.
—
Reply to this email directly, view it on GitHub<#513 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AX7UCAXOAMDAPHMI2JQSBDDYTF5SBAVCNFSM6AAAAABDCFOZQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZXHE4TKNRQGQ>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
Hi, Thank you for showing interest in this project. The issue labeled as "good first issue" is based on the assumption that addressing it won't likely have a significant impact on the overall project, as mentioned in the issue description. As stated in the issue, Therefore, there's likely no need to worry about the proposed changes adversely affecting the project. Your contributions are welcome. For areas that you may not understand, I trust that the community will provide assistance. And I hope that this project proves to be helpful for you in obtaining credits. |
Alright then, I can open them back up. I will keep that in mind in case I try and tackle any other task! Thanks so much.
…________________________________
From: Jun Komoda ***@***.***>
Sent: Monday, February 12, 2024 9:29 AM
To: enthought/comtypes ***@***.***>
Cc: Calvo, Alexander J. ***@***.***>; State change ***@***.***>
Subject: Re: [enthought/comtypes] Deleted __cmp__() functions from repository (PR #513)
[External Email]
Hi,
Thank you for showing interest in this project.
The issue labeled as "good first issue" is based on the assumption that addressing it won't likely have a significant impact on the overall project, as mentioned in the issue description.
As stated in the issue, __cmp__ and cmp belong to the older world of Python and are no longer in use. Projects explicitly implementing cmp in Python 3.7 or later are probably rare. If such projects exist, I would suggest to them to use __eq__ instead.
Therefore, there's likely no need to worry about the proposed changes adversely affecting the project.
Your contributions are welcome. For areas that you may not understand, I trust that the community will provide assistance.
And I hope that this project proves to be helpful for you in obtaining credits.
—
Reply to this email directly, view it on GitHub<#513 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AX7UCAUW6QNMBPB27LBWJ3DYTIRM7AVCNFSM6AAAAABDCFOZQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZYG44DKMJYGQ>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
Here I deleted the out of data cmp() functions in comtypes as stated in issue #512 as it is no longer supported