-
Notifications
You must be signed in to change notification settings - Fork 190
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
enable deepcopy on PyObject #757 #1039
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1039 +/- ##
==========================================
+ Coverage 67.03% 67.52% +0.49%
==========================================
Files 20 20
Lines 2017 2023 +6
==========================================
+ Hits 1352 1366 +14
+ Misses 665 657 -8
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Why not just increment the refcount as proposed in #757? Or call Python's |
Co-authored-by: Steven G. Johnson <stevenj@mit.edu>
@stevengj Does this PR look good to you? |
@stevengj LGTY? |
Thanks, @stevengj can you tag a release? |
Will do |
No description provided.