zc he

Results 139 comments of zc he

> I don't see how metadata is the savior here, I think it would require explicit pre-verified `Value::Table` and `PipelineData::TableStream` to have guaranteed info about the valid columns. (as well...

> @blindFS Could you move these changes to the External Completers page in the Cookbook, as well as rename that page to Completions or Completers? Sorry, I don't have my...

Ideally, highlighting better to be an async function and then we can set a debounce timeout for it. I'm not sure if it worth the effort, since the only remaining...

Need more test in different environments

Moving cursor around (left/right mostly) also triggers the repainting, prompt + parsing + highlighting. I think that could also be optimized.

> @blindFS Is this ready to review? It's too hacky and of low priority. I'd prefer an async solution for all events anyways.

Thanks for the detailed explanations, closing.

In terms of semantic accuracy/completeness, I think it's better to keep as it is. Maybe because I never dig deep into the nushell's parser, the boundary between internal commands and...

I overlooked the type column I guess, it's hidden behind `...` XD. Some of the rules that are not in the list of `scope commands | where type == keyword`...