Skip to content

Conversation

@encukou
Copy link
Member

@encukouencukou commented Aug 10, 2022

This is the non-refactoring part of #95505.

(Plus the ACKS fixup I forgot to include in #95819 😅)

Copy link
Contributor

@philg314philg314 left a comment

Choose a reason for hiding this comment

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

There are some minor adjustment to the news entries based on the news entries for the %% changes but other than that everything is in order. Thank you!

encukouand others added 2 commits August 10, 2022 11:04
Co-authored-by: philg314 <110174000+philg314@users.noreply.github.com>
@encukouencukou merged commit 71c3d64 into python:mainAug 10, 2022
@encukouencukou deleted the unicode-from-format-negative branch August 10, 2022 11:12
copied as-is to the result string, and any extra arguments discarded.
(Contributed by Serhiy Storchaka in :gh:`95781`.)

* Fixed wrong sign placement in :c:func:`PyUnicode_FromFormat` and
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry!

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

Oh, no problem! I'm just not a fan of the nitpycky tests :)

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.

3 participants

@encukou@philg314@bedevere-bot