fix: workspace/executeCommand with async handlers

[alcarney]

Description (e.g. "Related to ...", etc.)

Turns out I forgot to handle async command handlers when implementing #527!

When wrapping an async handler with the language server instance, set the func attribute on the resulting handler. This allows the code that processes the result of functools.partial() to be reused. This ensures the workspace/executeCommand code can obtain the correct function signature to map values onto.

Code review checklist (for code reviewer to complete)

• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
• Title summarizes what is changing
• Commit messages are meaningful (see this for details)
• Tests have been included and/or updated, as appropriate
• Docstrings have been included and/or updated, as appropriate
• Standalone docs have been updated accordingly

Automated linters

You can run the lints that are run on CI locally with:

poetry install --all-extras --with dev
poetry run poe lint

[tombh]

Was the Windows build just flaking?

[alcarney]

Thanks!

Was the Windows build just flaking?

Yes... not sure what was going on there, the test suite got stuck for some reason and the job timed out. Something to keep an eye on I guess.