You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
However, I don't think that it should be. It's a complete implementation of IHelpWriter that can work on its own. Now we're forcing people to extend it, just to call the constructor of the abstract base class PrefixedKeyConsoleHelpWriter. It's clean, because it forces strong-typed help writers, but it also makes introducing a new one for a 3rd-party thing much more involved.
Suggestion: Remove the abstract modified from PrefixedKeyConsoleHelpWriter.
The text was updated successfully, but these errors were encountered:
Currently,
PrefixedKeyConsoleHelpWriter
is an abstract class:AtleX.CommandLineArguments/src/AtleX.CommandLineArguments/Help/PrefixedKeyConsoleHelpWriter.cs
Line 8 in cdf8fc5
However, I don't think that it should be. It's a complete implementation of
IHelpWriter
that can work on its own. Now we're forcing people to extend it, just to call the constructor of the abstract base classPrefixedKeyConsoleHelpWriter
. It's clean, because it forces strong-typed help writers, but it also makes introducing a new one for a 3rd-party thing much more involved.Suggestion: Remove the
abstract
modified fromPrefixedKeyConsoleHelpWriter
.The text was updated successfully, but these errors were encountered: