Skip to content

Fix trace param interface names #1142

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

Merged
merged 1 commit into from
Nov 16, 2020
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions _specifications/specification-3-16.md
Original file line number Diff line number Diff line change
Expand Up @@ -2193,17 +2193,17 @@ _Notification_:
A notification to log the trace of the server's execution.
The amount and content of these notifications depends on the current `trace` configuration.
If `trace` is `'off'`, the server should not send any `logTrace` notification.
If `trace` is `'message'`, the server should not add the `'verbose'` field in the `logTraceParams`.
If `trace` is `'message'`, the server should not add the `'verbose'` field in the `LogTraceParams`.

`$/logTrace` should be used for systematic trace reporting. For single debugging messages, the server should send [`window/logMessage`](#window_logMessage) notifications.


_Notification_:
* method: '$/logTrace'
* params: `logTraceParams` defined as follows:
* params: `LogTraceParams` defined as follows:

```typescript
interface logTraceParams {
interface LogTraceParams {
/**
* The message to be logged.
*/
Expand All @@ -2222,10 +2222,10 @@ A notification that should be used by the client to modify the trace setting of

_Notification_:
* method: '$/setTrace'
* params: `setTraceParams` defined as follows:
* params: `SetTraceParams` defined as follows:

```typescript
interface setTraceParams {
interface SetTraceParams {
/**
* The new value that should be assigned to the trace setting.
*/
Expand Down