command-line-api icon indicating copy to clipboard operation
command-line-api copied to clipboard
verified publisher icon dotnet

List<CliToken> tokenList should be renamed to List<CliToken> tokens

Open PonchoPowers opened this issue 1 year ago • 0 comments

There are a few mentions of tokenList left in Tokenizer.Tokenize and they should be renamed to remove the type "List" so that the naming conforms to the conventions of the .NET Framework.

Also, where (CliSymbol Symbol, CliTokenType TokenType)> tokens is used, the word tokens should be changed to validTokens to avoid any confusion between the list of valid tokens and the list of tokens being built up.

PonchoPowers avatar Jun 07 '24 11:06 PonchoPowers