Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Undo pytest memory bandage. #216

Merged
merged 7 commits into from
Feb 18, 2021
Merged

Undo pytest memory bandage. #216

merged 7 commits into from
Feb 18, 2021

Conversation

MTCam
Copy link
Member

@MTCam MTCam commented Feb 4, 2021 •
edited
Loading

Just a placeholder CI test to demonstrate the issue until #212 resolution.

@MTCam MTCam marked this pull request as ready for review February 5, 2021 00:51
This was referenced Feb 9, 2021
Closed
Merged
@MTCam MTCam enabled auto-merge (squash) February 10, 2021 13:49
@MTCam MTCam disabled auto-merge February 10, 2021 13:50
@inducer inducer mentioned this pull request Feb 10, 2021
Draft
inducer
inducer approved these changes Feb 17, 2021
View reviewed changes
Copy link
Contributor

@inducer inducer left a comment

Choose a reason for hiding this comment

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

I don't know why this passes now, but I'm in favor of merging it anyhow. Let me re-run the CI to see how lucky we are. But LGTM at any rate.

@MTCam MTCam merged commit b7d29a1 into master Feb 18, 2021
@MTCam MTCam deleted the undo-individual-pytest branch February 18, 2021 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthiasdiener matthiasdiener matthiasdiener approved these changes

@inducer inducer inducer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MTCam @inducer @matthiasdiener