-
Notifications
You must be signed in to change notification settings - Fork 481
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
Added support for Auto Labeling to SCSensitivityLabel and some other minor fixes #5062
base: Dev
Are you sure you want to change the base?
Conversation
@@ -108,11 +108,6 @@ | |||
ModuleName = 'PnP.PowerShell' | |||
RequiredVersion = '1.12.0' | |||
}, | |||
@{ | |||
ModuleName = 'PSDesiredStateConfiguration' | |||
RequiredVersion = '1.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why removing this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is because PSDesiredStateConfiguration isn't available in the PSGallery, but is installed by default with PSv5.1. When running the Update Dependencies cmdlet, this will never install because it is always present.....however when you have broken your environment the function cannot fix it because the module cannot be downloaded from the Gallery anyways.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v1.1 that is
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bottom line, are we okay with removing this from the manifest? @ricmestre
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NikCharlebois Yes, let's remove this please
Pull Request (PR) description
This PR adds support for Auto Labeling settings to the SCSensitivityLabel resource, but also fixes some other minor issues I have encountered during testing.
This Pull Request (PR) fixes the following issues