Syntax highlighting for match and case

[cdce8p]

It would be nice if syntax highlighting for match and case could be added.

def http_error(status: int):
    match status:
        case 400:
            return "Bad request"
        case 404:
            return "Not found"
        case 418:
            return "I'm a teapot"

[jakebailey]

Thanks for the report; at the moment we rely on the TextMate profile built into VS Code to display these kinds of keywords. I'm surprised they haven't been added there, but I do think we should be emitting keywords as tokens in the semantic highlighter as well.

[cdce8p]

After searching a bit more, I found these two existing issues:
microsoft/vscode#120734
MagicStack/MagicPython#235

I've also opened a PR MagicStack/MagicPython#237 to recognize match and case as keywords, at least in a match statement.

[jakebailey]

In the meantime while we wait for VS Code and such to pull in those changes, I've modified our semantic tokenizer to output match and case as keyword tokens:

[cdce8p]

@jakebailey That's great! Thanks 🚀

[jakebailey]

This issue has been fixed in version 2021.5.0, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202150-5-may-2021

[Jaakkonen]

For people using pyright extension for development of it I think there's no other workaround than using a MagicPython extension built from the PR branch (rebased or not).

[scarf005]

default themes and github themes support match case syntax, however majority of other themes does not support them out of the box.

running Inspect editor tokens and scopes shows result like this:

could there be any way to overwrite custom themes to support match case?
(using editor.tokenColorCustomizations for every single themes would too much work)

[debonte]

@scarf005, this issue has been closed for a long time. Please open a new issue for your suggestion.

[scarf005]

@scarf005, this issue has been closed for a long time. Please open a new issue for your suggestion.

alright, referenced in new issue.