Skip to content
This repository has been archived by the owner. It is now read-only.

feat(config): Merge with existing config when writing to memcache. #151

Merged
merged 1 commit into from Jan 19, 2017

Conversation

@rfk
Copy link
Member

@rfk rfk commented Nov 24, 2016

This makes it easier to change a single setting while leaving all the others alone. Fixes #127. @jrgm r?

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Dec 12, 2016

from mtg: @jrgm this needs your input

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Jan 9, 2017

@rfk can we rebase this?

This makes it easier to change a single setting while leaving all
the others alone.
@rfk rfk force-pushed the config-set-merge branch from 6e5ae76 to f199570 Jan 10, 2017
@rfk
Copy link
Member Author

@rfk rfk commented Jan 10, 2017

There were flaky tests here that seem unrelated to this PR; I restarted the jobs and they're green. @jrgm final r?

@rfk rfk removed their assignment Jan 10, 2017
@vladikoff vladikoff merged commit a8f4d68 into master Jan 19, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 94.852%
Details
@vladikoff vladikoff deleted the config-set-merge branch Jan 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants