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

Don’t delete properties from mixin objects passed to .extend #111

Merged
merged 1 commit into from Nov 26, 2014

Conversation

Projects
None yet
6 participants
@latentflip
Contributor

latentflip commented Nov 26, 2014

Fixes: #68, #110

@bear bear added the request label Nov 26, 2014

@mmacaula

This comment has been minimized.

Show comment
Hide comment
@mmacaula

mmacaula Nov 26, 2014

Contributor

Awesome @latentflip thanks for a quick turnaround on this!

Contributor

mmacaula commented on 3a1baef Nov 26, 2014

Awesome @latentflip thanks for a quick turnaround on this!

@kamilogorek

This comment has been minimized.

Show comment
Hide comment
@kamilogorek
Member

kamilogorek commented Nov 26, 2014

+1

@lukekarrys

This comment has been minimized.

Show comment
Hide comment
@lukekarrys

lukekarrys Nov 26, 2014

Contributor

+1

Contributor

lukekarrys commented Nov 26, 2014

+1

@HenrikJoreteg

This comment has been minimized.

Show comment
Hide comment
@HenrikJoreteg
Member

HenrikJoreteg commented Nov 26, 2014

+1

latentflip added a commit that referenced this pull request Nov 26, 2014

Merge pull request #111 from AmpersandJS/fix-110
Don’t delete properties from mixin objects passed to .extend

@latentflip latentflip merged commit 8fb69a8 into master Nov 26, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@latentflip latentflip deleted the fix-110 branch Nov 26, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment