Skip to content

feat(config): adds support for top level file includes#112

Open
DelphicOkami wants to merge 4 commits into
Adembc:mainfrom
DelphicOkami:config.dSupport
Open

feat(config): adds support for top level file includes#112
DelphicOkami wants to merge 4 commits into
Adembc:mainfrom
DelphicOkami:config.dSupport

Conversation

@DelphicOkami
Copy link
Copy Markdown

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 file element in metadata.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 Match directive are not mapped to any hosts during config editing

@DelphicOkami
Copy link
Copy Markdown
Author

I know this solves the same issue as #76 but I wanted more comprehensive support

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant