Skip to content

[3.10] bpo-44116: Add GC support to _csv heap types (GH-26074) #26081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 12, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 12, 2021

(cherry picked from commit e5ba1fe)

Co-authored-by: Erlend Egeberg Aasland [email protected]

https://bugs.python.org/issue44116

(cherry picked from commit e5ba1fe)

Co-authored-by: Erlend Egeberg Aasland <[email protected]>
@miss-islington
Copy link
Contributor Author

@erlend-aasland and @pablogsal: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@erlend-aasland and @pablogsal: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@erlend-aasland and @pablogsal: Status check is done, and it's a success ✅ .

@erlend-aasland
Copy link
Contributor

Can I add the automerge label, @pablogsal?

@pablogsal
Copy link
Member

Backports are automatically merged when all CI passed if a core dev approves :)

@erlend-aasland
Copy link
Contributor

Backports are automatically merged when all CI passed if a core dev approves :)

Convenient! :)

@erlend-aasland
Copy link
Contributor

Looks like Miss Islington needs a bugfix ;) #automerge

@pablogsal pablogsal merged commit ba260ac into python:3.10 May 12, 2021
@miss-islington miss-islington deleted the backport-e5ba1fe-3.10 branch May 12, 2021 18:56
@pablogsal
Copy link
Member

CC: @Mariatta

Seems that Miss islington didn't merge this PR even if all test passed :(

@erlend-aasland
Copy link
Contributor

Perhaps it is intentional: python/miss-islington#253

(I just skimmed the "status changed" code, so I might be wrong.)

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.

5 participants