Skip to content

Support copy of custom classes#430

Merged
jakebeal merged 4 commits intomainfrom
copy-custom-toplevel
Apr 11, 2023
Merged

Support copy of custom classes#430
jakebeal merged 4 commits intomainfrom
copy-custom-toplevel

Conversation

@jakebeal
Copy link
Copy Markdown
Contributor

@jakebeal jakebeal commented Apr 7, 2023

Copying a CustomTopLevel or CustomIdentified without a registered builder would cause an error. Instead, we can fall back onto using a CustomTopLevel or CustomIdentified, just as we do when loading the document in the first place. Test has been extended to check the case of unregistered custom objects for both round tripping and copying

Copying a CustomTopLevel or CustomIdentified without a registered builder would cause an error.
Instead, we can fall back onto using a CustomTopLevel or CustomIdentified, just as we do when loading the document in the first place.
Test has been extended to check the case of unregistered custom objects for both round tripping and copying
@jakebeal jakebeal requested a review from tcmitchell April 7, 2023 18:18
Comment thread test/test_custom.py Outdated
Comment thread test/test_custom.py Outdated
@jakebeal jakebeal merged commit c84ccd1 into main Apr 11, 2023
@jakebeal jakebeal deleted the copy-custom-toplevel branch April 11, 2023 14:15
@tcmitchell tcmitchell added this to the 1.1 milestone Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants