Skip to content

Conversation

@kj7rrv
Copy link
Contributor

@kj7rrvkj7rrv commented Jul 5, 2022

@ghost
Copy link

ghost commented Jul 5, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@arhadthedev
Copy link
Member

arhadthedev commented Jul 5, 2022

I tried to do the same fix in gh-92560. Currently it stuck on determination whether developer-only tools should be tested, so I'm porting this function to python/pyperformance (pprint branch of my fork - I can't create a PR until I fix some bugs).

@kj7rrv
Copy link
ContributorAuthor

@arhadthedev should I close this?

@arhadthedev
Copy link
Member

@kj7rrv I would wait until a triager or a core developer walks by. Maybe your PR will be accepted so I'll close mine.

@kj7rrv
Copy link
ContributorAuthor

@arhadthedev Oh, okay. I'll leave it.

@kj7rrv
Copy link
ContributorAuthor

How long do the tests generally take?

@arhadthedev
Copy link
Member

The port is done: python/pyperformance#222. You can track it to close your PR after/if it's merged.

@kj7rrv
Copy link
ContributorAuthor

So I should change this PR to remove _perfcheck instead of fixing the bug? @arhadthedev

@kj7rrv
Copy link
ContributorAuthor

Superseded by gh-94613

@kj7rrvkj7rrv closed this Jul 6, 2022
@kj7rrvkj7rrv deleted the fix-pprint branch July 6, 2022 19:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minor bug in pprint module

3 participants

@kj7rrv@bedevere-bot@arhadthedev