-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Implement Group Policy #810
Conversation
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
Co-authored-by: Josh Soref <[email protected]>
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
This is still missing E2E tests and the policies for controlling sources. I'll submit that as a separate PR as this one already got too big. |
@@ -87,14 +87,27 @@ namespace AppInstaller::Registry | |||
template <Type T> | |||
typename details::ValueTypeSpecifics<static_cast<DWORD>(T)>::value_t GetValue() const | |||
{ | |||
EnsureType(T); | |||
THROW_HR_IF(E_INVALIDARG, !HasCompatibleType(T)); |
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.
THROW_HR_IF(E_INVALIDARG, !HasCompatibleType(T)); [](start = 12, length = 49)
nit (for now): Better to implement by calling TryGetValue and throwing when it returns nullopt to prevent future changes from needing to be made to both code paths.
Implementing group policy. Interface is roughly based on ExperimentalFeatures and UserSettings.
This is still a draft. Still pending:
Related: #809 #154
Microsoft Reviewers: Open in CodeFlow