gh-101152: Implement PEP 699 by ramvikrams · Pull Request #101193 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
Conversation
- Please add news.
- The code looks similar to Kumar's. I'll add him as co-author when I merge.
added the news and infact it is a replica of his solution
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some feedback for your future contributions:
- You can close conversations. Click on the "resolve conversation" button once you feel that you resolved the comment.
- reStructuredText markup is a little tricky! So you can refer to the devguide for that https://devguide.python.org/documentation/markup/
- I'll try to summarise what I look for in the Whats New deprecation. First you want to know exactly what is being affected (in this case the field of a certain struct being deprecated). Secondly, you want to know, without ambiguity, how we will notify users of this deprecation. In this case it's a compiler warning. Lastly, if there's a planned removal version for the deprecated thing, it should be mentioned too.
Thank you.
Fidget-Spinner
changed the title
gh-101152: Added the PEP 699 Implementation
gh-101152: Implement PEP 699
Just some feedback for your future contributions:
The feedback was very helpful, will surely keep the points in mind.
Thanks
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can you rebase with main and fix the conflicts in 3.12 whats new please? Seems like someone wrote an entry at the current spot.
Can you rebase with main and fix the conflicts in 3.12 whats new please? Seems like someone wrote an entry at the current spot.
yes sure