◐ Shell
clean mode source ↗

gh-101152: Implement PEP 699 by ramvikrams · Pull Request #101193 · python/cpython

Skip to content

Navigation Menu

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@Fidget-Spinner

  1. Please add news.
  2. The code looks similar to Kumar's. I'll add him as co-author when I merge.

@ramvikrams

  1. Please add news.
  2. 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

Fidget-Spinner

Fidget-Spinner

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:

  1. You can close conversations. Click on the "resolve conversation" button once you feel that you resolved the comment.
  2. reStructuredText markup is a little tricky! So you can refer to the devguide for that https://devguide.python.org/documentation/markup/
  3. 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 Fidget-Spinner changed the title gh-101152: Added the PEP 699 Implementation gh-101152: Implement PEP 699

Jan 21, 2023
Co-authored-by: Ken Jin <kenjin@python.org>
…8pL.rst

Co-authored-by: Ken Jin <kenjin@python.org>

@ramvikrams

Just some feedback for your future contributions:

The feedback was very helpful, will surely keep the points in mind.

Thanks

Fidget-Spinner

Fidget-Spinner

Co-authored-by: Ken Jin <kenjin@python.org>

kumaraditya303

Choose a reason for hiding this comment

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

LGTM

@Fidget-Spinner

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.

@ramvikrams

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

Fidget-Spinner