Skip to content

gh-92546: Move pprint benchmark into pyperformance - #94613

Merged
miss-islington merged 4 commits into
python:mainfrom
arhadthedev:move-pprint-benchmark
Jul 25, 2022
Merged

gh-92546: Move pprint benchmark into pyperformance#94613
miss-islington merged 4 commits into
python:mainfrom
arhadthedev:move-pprint-benchmark

Conversation

@arhadthedev

@arhadthedevarhadthedev commented Jul 6, 2022

Copy link
Copy Markdown
Member

This PR couples with python/pyperformance#222 and supersedes #92560. Inspired by #93096 (comment).

Automerge-Triggered-By: GH:ericsnowcurrently

@hauntsaninjahauntsaninja left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Had a nit, but looks good!

Comment threadMisc/NEWS.d/next/Library/2022-07-06-21-24-03.gh-issue-92546.s5Upkh.rst Outdated
Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
@arhadthedev

Copy link
Copy Markdown
MemberAuthor

Restart tests because test.test_ssl.SimpleBackgroundTests.test_get_server_certificate_timeout timeout failure seems unrelated.

@arhadthedev

Copy link
Copy Markdown
MemberAuthor

@ericsnowcurrently Could you look at this PR please? It removes the code fetched by python/pyperformance#222.

@ericsnowcurrentlyericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, aside from one small change.

Comment threadMisc/NEWS.d/next/Library/2022-07-06-21-24-03.gh-issue-92546.s5Upkh.rst Outdated
@miss-islington

Copy link
Copy Markdown
Contributor

Status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit c514094 into python:mainJul 25, 2022
@arhadthedev
arhadthedev deleted the move-pprint-benchmark branch July 25, 2022 18:32
@ericsnowcurrently

Copy link
Copy Markdown
Member

Thanks for working on this, @arhadthedev!

Sign up for freeto 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

@arhadthedev@miss-islington@ericsnowcurrently@hauntsaninja@bedevere-bot