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

[PR #6117/3862de3f backport][stable-6] Removes dependency on StormSSH #6179

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Mar 12, 2023

This is a backport of PR #6117 as merged into main (3862de3).

SUMMARY

Pull in StormSSH's config parser into ssh_config
Fixes #6009

ISSUE TYPE
  • Refactoring Pull Request
COMPONENT NAME

ssh_config

ADDITIONAL INFORMATION

This PR simply pulls in the config parser from the now archived StormSSH into ssh_config itself.

* included storm config parser

* Add changelog fragment

* Fix changelog fragment

* address PR feedback

* fix license

* add required boilerplate, fix issues found in test output

* move __future__ imports

* address pr feedback

* address test output

* address pr feedback

(cherry picked from commit 3862de3)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport integration tests/integration module module module_utils module_utils new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests labels Mar 12, 2023
@felixfontein felixfontein merged commit 8e84b3e into stable-6 Mar 12, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/3862de3f153c4401b7d77dba9e71a6535973ecb4/pr-6117 branch March 12, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration module_utils module_utils module module new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants