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

Config import partial #1170

Merged
merged 3 commits into from
Mar 23, 2015
Merged

Config import partial #1170

merged 3 commits into from
Mar 23, 2015

Conversation

jhedstrom
Copy link
Member

This is a start on #1163. It shouldn't be too much more work to allow for directory imports of partial config sets.

@jhedstrom
Copy link
Member Author

Failure looks to be unrelated (honeypot needs updating since _l() was removed from 8.x).

@weitzman
Copy link
Member

Looking good so far. Thanks for the test.

@weitzman
Copy link
Member

I really like the --partial feature for config-import. I'm ambivalent on config-edit --file but if you think its valuable lets go with it. This passed tests so I think it can go in once @jhedstrom is happy with it.

@jhedstrom jhedstrom force-pushed the config-import-partial branch from 8a61268 to 9e61660 Compare March 23, 2015 20:23
jhedstrom added a commit that referenced this pull request Mar 23, 2015
@jhedstrom jhedstrom merged commit 1201d57 into master Mar 23, 2015
@jhedstrom
Copy link
Member Author

I've left the config-edit --file option in place for now.

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

Successfully merging this pull request may close these issues.

2 participants