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.

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

remove the memleak tests #10322

Merged
merged 1 commit into from
Feb 2, 2024
Merged

remove the memleak tests #10322

merged 1 commit into from
Feb 2, 2024

Conversation

reaperhulk
Copy link
Member

@reaperhulk reaperhulk commented Feb 2, 2024

they are fragile, haven't caught regressions, and are increasingly pointless as we oxidize.

they are fragile, haven't caught regressions, and increasingly pointless
as we oxidize.
@alex alex enabled auto-merge (squash) February 2, 2024 02:29
@alex alex merged commit c0c9ec8 into pyca:main Feb 2, 2024
57 checks passed
@reaperhulk reaperhulk deleted the memleak-gone branch February 2, 2024 02:39
reaperhulk added a commit to reaperhulk/cryptography that referenced this pull request Feb 3, 2024
they are fragile, haven't caught regressions, and increasingly pointless
as we oxidize.
alex pushed a commit that referenced this pull request Feb 3, 2024
* initialize openssl's legacy provider in rust (#10323)

* initialize openssl's legacy provider in rust

as we oxidize we need to do this here to ensure it actually happens

* alex is a comment format pedant

* remove the memleak tests (#10322)

they are fragile, haven't caught regressions, and increasingly pointless
as we oxidize.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants