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

fix(utils.extend): removes merge and makes extend behave #895

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@runspired
Contributor

runspired commented Dec 23, 2015

Previous fixes to prevent configuration leaking utilized extend and merge abnormally, both of which did not behave conventionally, leading to accidental overwriting or removal of user supplied configuration settings.

This deprecates extend and merge in favor of assign and removes their use internally (but leaves them publicly exposed).

@arschmitz

This comment has been minimized.

Show comment
Hide comment
@arschmitz

arschmitz Dec 24, 2015

Contributor

merged

Contributor

arschmitz commented Dec 24, 2015

merged

@arschmitz arschmitz closed this Dec 24, 2015

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