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

Unclonable content #65

Merged
merged 3 commits into from
Jun 19, 2022
Merged

Unclonable content #65

merged 3 commits into from
Jun 19, 2022

Conversation

igneus
Copy link
Contributor

@igneus igneus commented Aug 11, 2017

Examples like this

  • TreeNode.new('name', :symbol).detached_copy
  • TreeNode.new('name', 5).detached_copy
  • TreeNode.new('name', true).detached_copy

currently fail, with attempt to clone an unclonable object raising TypeError. This PR adds handling of unclonable content.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 98.293% when pulling d45517d on igneus:unclonable_content into db48c35 on evolve75:master.

@evolve75
Copy link
Owner

@igneus, thank you for this PR and fixing this bug. Much appreciated!

@evolve75 evolve75 merged commit 665b783 into evolve75:master Jun 19, 2022
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.

3 participants