Skip to content

bpo-1635741: Port statistics module to multiphase initialization#19015

Merged
vstinner merged 2 commits intopython:masterfrom
corona10:bpo-1635741-statistics
Mar 16, 2020
Merged

bpo-1635741: Port statistics module to multiphase initialization#19015
vstinner merged 2 commits intopython:masterfrom
corona10:bpo-1635741-statistics

Conversation

@corona10
Copy link
Member

@corona10 corona10 commented Mar 15, 2020

@corona10
Copy link
Member Author

@shihai1991 Can you please take a look?

Copy link
Member

@shihai1991 shihai1991 left a comment

Choose a reason for hiding this comment

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

I have checked this source file(there have no _Py_IDENTIFIER or global vars), so this PR LGTM, thanks.

@shihai1991
Copy link
Member

let us ping experts @vstinner @encukou @ncoghlan

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM, but please fix the typo in the commit message and in the NEWS entry: it's the private _statistics module, not statistics :-)

…5741.iH0JND.rst

Co-Authored-By: Victor Stinner <vstinner@python.org>
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.

5 participants