Skip to content

Commit

Permalink
fix pr comment rm build flash telemetry (#1328)
Browse files Browse the repository at this point in the history
  • Loading branch information
brianignacio5 authored Oct 23, 2024
1 parent ed521c9 commit f5ecca2
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pr-comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
return core.error(`No artifacts found`);
}
let body = `Download the artifacts for this pull request:\n`;
body += `You can test these changes by installing this VSIX by click menu `View` -> `Command Palette...`, type `Install from VSIX` and then select downloaded `esp-idf-extension.vsix` file to install the extension.`;
body += `You can test these changes by installing this VSIX by click menu **View** -> **Command Palette...**, type **Install from VSIX** and then select downloaded **esp-idf-extension.vsix** file to install the extension.`;
for (const art of artifacts) {
body += `\n* [${art.name}.zip](https://nightly.link/${owner}/${repo}/actions/artifacts/${art.id}.zip)`;
}
Expand Down
4 changes: 3 additions & 1 deletion src/build/buildCmd.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,9 @@ export async function buildCommand(
Logger.errorNotify(
"Something went wrong while trying to build the project",
error,
"buildCommand"
"buildCommand",
undefined,
false
);
continueFlag = false;
}
Expand Down
8 changes: 7 additions & 1 deletion src/flash/uartFlash.ts
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,13 @@ export async function flashCommand(
}
const errStr = `Failed to flash because of some unusual error. Check Terminal for more details`;
OutputChannel.appendLine(errStr, "Flash");
Logger.errorNotify(errStr, error, "flashCommand unknown error");
Logger.errorNotify(
errStr,
error,
"flashCommand unknown error",
undefined,
false
);
continueFlag = false;
}
FlashTask.isFlashing = false;
Expand Down

0 comments on commit f5ecca2

Please sign in to comment.