Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Jul 25, 2025

This bit wasn't too clear to me:

The code that as the time of writing this is in profile.sampling module no longer exist and will only exist in the profilers package.

This PR suggests:

The code that, at the time of writing, is in the profile.sampling module will be moved to the profilers package.

Is that the idea?

Alternatively:

The code that, at the time of writing, is in the profile.sampling module will no longer exist and will instead only exist in the profilers package.


📚 Documentation preview 📚: https://pep-previews--4511.org.readthedocs.build/

@hugovk hugovk requested review from a team and pablogsal as code owners July 25, 2025 11:30
@pablogsal
Copy link
Member

LGTM

@pablogsal pablogsal merged commit 2a0363a into python:main Jul 25, 2025
6 checks passed
@hugovk hugovk deleted the pep799-clarifications branch July 25, 2025 12:11
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.

2 participants