Skip to content

Conversation

@corona10
Copy link
Member

@corona10 corona10 commented Mar 4, 2022

@kumaraditya303
Copy link
Contributor

Wrong bpo number ?

Copy link
Contributor

@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

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

Check bpo number; else looks good to me.

Copy link
Contributor

@kumaraditya303 kumaraditya303 left a comment

Choose a reason for hiding this comment

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

LGTM

@corona10 corona10 changed the title bpo-46430: Remove usage of _Py_IDENTIFIER from lzma module bpo-46541: Remove usage of _Py_IDENTIFIER from lzma module Mar 4, 2022
@corona10
Copy link
Member Author

corona10 commented Mar 4, 2022

I will merge this PR once the adding label is available, it looks like the outage situation.

Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to 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.

6 participants