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

Make copying calculators optional #80

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

hdoupe
Copy link
Collaborator

@hdoupe hdoupe commented Mar 30, 2021

More info in this comment: #70 (comment)

cc @jdebacker

@hdoupe hdoupe mentioned this pull request Apr 6, 2021
@hdoupe
Copy link
Collaborator Author

hdoupe commented Apr 8, 2021

I merged the branch from #82 into this one and pushed it. So the ci is running here now. I'm sorry for the confusion but at least it works now?

@jdebacker
Copy link
Member

@hdoupe Is this PR ready to go?

@MattHJensen any comments?

@hdoupe
Copy link
Collaborator Author

hdoupe commented Apr 15, 2021

@jdebacker , yes it is.

@MattHJensen this is ready for your review.

@MattHJensen MattHJensen changed the title [WIP] Make copying calculators optional Make copying calculators optional Apr 15, 2021
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.

None yet

2 participants