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

Copy Attributes when copy is used. #437

Merged
merged 1 commit into from Nov 2, 2017

Conversation

Projects
None yet
2 participants
@rainwoodman
Member

rainwoodman commented Nov 2, 2017

Do not copy attributes when view is used.

Copy Attributes when copy is used.
Do not copy attributes when view is used.

@rainwoodman rainwoodman requested a review from nickhand Nov 2, 2017

@rainwoodman

This comment has been minimized.

Member

rainwoodman commented Nov 2, 2017

I was meddling with BoxSize of a copied catalog and realized it would have changed the original BoxSize..

@nickhand

looks good to me, I'll merge

@nickhand nickhand merged commit 5fd0074 into bccp:master Nov 2, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.0008%) to 95.345%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment