Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow configuring debugExpressionTimeoutMs #613

Conversation

axelson
Copy link
Member

@axelson axelson commented Oct 20, 2021

Rather than a hard-coded timeout of 1_000 ms for watch expressions, add
debugExpressionTimeoutMs to the launch.json configuration options. In
the next version of vscode-elixir-ls debugExpressionTimeoutMs will be
supported in the launch.json autocomplete, but if you are running a
version of ElixirLS that supports it before then you can add the key
manually.

Example mix test launch.json entry that uses this to set a timeout of 30
seconds:

{
    "type": "mix_task",
    "name": "mix test",
    "request": "launch",
    "debugExpressionTimeoutMs": 30000,
    "task": "test",
    "taskArgs": [
        "--trace"
    ],
    "startApps": true,
    "projectDir": "${workspaceRoot}",
    "requireFiles": [
        "test/**/test_helper.exs",
        "test/**/*_test.exs"
    ]
}

Fixes #525

Rather than a hard-coded timeout of 1_000 ms for watch expressions, add
`debugExpressionTimeoutMs` to the launch.json configuration options. In
the next version of vscode-elixir-ls `debugExpressionTimeoutMs` will be
supported in the launch.json autocomplete, but if you are running a
version of ElixirLS that supports it before then you can add the key
manually.

Example mix test launch.json entry that uses this to set a timeout of 30
seconds:
```
{
    "type": "mix_task",
    "name": "mix test",
    "request": "launch",
    "debugExpressionTimeoutMs": 30000,
    "task": "test",
    "taskArgs": [
        "--trace"
    ],
    "startApps": true,
    "projectDir": "${workspaceRoot}",
    "requireFiles": [
        "test/**/test_helper.exs",
        "test/**/*_test.exs"
    ]
}
```

Fixes elixir-lsp#525
@lukaszsamson lukaszsamson merged commit 37dd730 into elixir-lsp:master Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add parameter to set evaluate_code_expression timeout
2 participants