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

Add new resource for configuring Wsman:\localhost\* items #38

Closed
PlagueHO opened this issue Sep 19, 2019 · 1 comment
Closed

Add new resource for configuring Wsman:\localhost\* items #38

PlagueHO opened this issue Sep 19, 2019 · 1 comment

Comments

@PlagueHO
Copy link
Member

The current resource WSManServiceConfig only configures settings in the wsman:\localhost\service* path. We should create a new resource that manages the items in the Wsman:\localhost* path called WSManConfig.

This would also imply a resource should be created called WSManClientConfig and WSManShellConfig.

These new resources would be copied from WSManServiceConfig with the data files changed. Although combining these properties into the existing WSManServiceConfig would be a simple approach too (with some modification) it would result in the name needing to be changed to correctly indicate the purpose of the resource - which would result in a breaking change.

So implementing multiple resources seems to be the best approach.

See https://github.com/PlagueHO/WSManDsc/issues/2#issuecomment-532773432

Tagging @fullenw1

@johlju - any thoughts on this one?

@PlagueHO
Copy link
Member Author

PlagueHO commented Oct 4, 2019

Will not do this one. Instead will separate this into separate resources.

@PlagueHO PlagueHO closed this as completed Oct 4, 2019
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

No branches or pull requests

1 participant