Respond to review

Author: KathleenDollard

src/System.CommandLine.Subsystems/Validation/InclusiveGroupValidator.cs

@@ -41,6 +41,7 @@ public override void Validate(CliCommandResult commandResult,
         }
         if (missingMembers is not null && missingMembers.Any())
         {
+            // TODO: Rework to allow localization
             var pluralToBe = "are";
             var singularToBe = "is";
             validationContext.PipelineResult.AddError(new ParseError( $"The members {string.Join(", ", groupMembers.Select(m => m.Name))} " +

src/System.CommandLine.Subsystems/Validation/Validator.cs

@@ -8,6 +8,7 @@ namespace System.CommandLine.Validation;
 /// <summary>
 /// Base class for CommandValidator and ValueValidator. 
 /// </summary>
+// TODO: Discuss visibility and custom validators
 public abstract class Validator
 {
     public Validator(string name, Type valueConditionType, params Type[] moreValueConditionTypes)