search

Jawz84 / explainpowershell

PowerShell version of explainshell.com
https://www.explainpowershell.com/
MIT License
34 stars 1 forks source link

The semicolon is not explained as a command separator #80

Closed LaurentDardenne closed 2 years ago

LaurentDardenne commented 2 years ago

With :

Dir;Cls

The presence of the comma is missing from the explanation.

Jawz84 commented 2 years ago

Good catch! I didn't even realize this was missing. It is also an interesting one, as the line separator is not part of the AST, where all explanations are handled. It is handled by the tokenizer. The result of the tokenizer is the basis for the AST, but since explanations are only generated by walking the AST, the ; sign does not get an explanation at this time. I agree this needs to be fixed.