feat(config): adds support for top level file includes#112
Open
DelphicOkami wants to merge 4 commits into
Open
feat(config): adds support for top level file includes#112DelphicOkami wants to merge 4 commits into
DelphicOkami wants to merge 4 commits into
Conversation
Author
|
I know this solves the same issue as #76 but I wanted more comprehensive support |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds support for top level include directives in ssh config (IE: those outside of any blocks), writes changes back to the file containing the host's definition. If the host is defined in multiple files the user is presented with a selector to choose the file to write back to, this choice is then persisted as a
fileelement inmetadata.json, when this key is present for a host we skip the selector.When writing changes back to an include file this ensures that only minimal changes are written back to prevent config duplication.
When adding new hosts this does not change the current behavior of writing them to the default ssh config.
Current limitations wildcard configuration and config using the
Matchdirective are not mapped to any hosts during config editing