fix(cli): Convert policy set --manual to use a tri-state #3349
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As per issue #3344, If Manual is set to true in a more-general policy, it cannot be reset to false in a lower policy. This is due to manual being a bool, and there being no way to distinguish between 'inherit' and 'false'
This PR converts Manual into an OptionalBool so that it can hold values of true, false, and inherit, and can thus override both from false->true and from true->false
There is an included test to validate as well.
This PR includes #3348 as inheritance cannot be properly tested until that issue is addressed. I recommend committing #3348 1st, and then this PR