Skip to content

Commit

Permalink
Don't show HHVM status by default if LSP doesn't send it over
Browse files Browse the repository at this point in the history
  • Loading branch information
PranayAgarwal committed Jul 26, 2019
1 parent 5f6700e commit cd97087
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/LSPHackTypeChecker.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,9 @@ export class LSPHackTypeChecker {
public async run(): Promise<void> {
const context = this.context;

this.status.text = "$(alert) " + this.versionText;
this.status.tooltip = "hh_server is not running for this workspace.";
this.status.show();
// this.status.text = "$(alert) " + this.versionText;
// this.status.tooltip = "hh_server is not running for this workspace.";
// this.status.show();

const serverOptions: ServerOptions = {
command: remote.getCommand(config.clientPath),
Expand Down Expand Up @@ -65,6 +65,7 @@ export class LSPHackTypeChecker {
this.status.text = "$(alert) " + this.status.text;
}

this.status.show();
return {};
});

Expand Down

0 comments on commit cd97087

Please sign in to comment.