Take kind instead of edit in CodeAction ctor

Changes the `CodeAction` constructor to take a kind instead of an edit. This makes the API more consistent IMO, as now both `edit` and `command` are set the same way
This commit is contained in:
Matt Bierner
2018-01-22 11:59:17 -08:00
parent 3cb83387be
commit 6316a4d0c8
5 changed files with 26 additions and 31 deletions

View File

@@ -417,10 +417,10 @@ export class ExtHostApiCommands {
} else {
const ret = new types.CodeAction(
codeAction.title,
typeConverters.WorkspaceEdit.to(codeAction.edit)
codeAction.kind ? new types.CodeActionKind(codeAction.kind) : undefined
);
if (codeAction.kind) {
ret.scope = new types.CodeActionKind(codeAction.kind);
if (codeAction.edit) {
ret.edit = typeConverters.WorkspaceEdit.to(codeAction.edit);
}
return ret;
}

View File

@@ -819,11 +819,11 @@ export class CodeAction {
dianostics?: Diagnostic[];
scope?: CodeActionKind;
kind?: CodeActionKind;
constructor(title: string, edit?: WorkspaceEdit) {
constructor(title: string, kind?: CodeActionKind) {
this.title = title;
this.edit = edit;
this.kind = kind;
}
}