github/codeql-action
0
0
Fork 0
mirror of https://github.com/github/codeql-action.git synced 2026-05-04 04:40:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Call core.endGroup() when doing an early return in installPythonDeps

Browse Source
This commit is contained in:
David Verdeguer
2020-07-23 13:05:30 +02:00
parent 4d4cc33d97
commit d190a018d6
3 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/setup-tracer.js
Generated
+2
View File
@@ -133,6 +133,7 @@ async function installPythonDeps(codeql) {
catch (e) {
// The download should not fail, but in case it fails we just abort trying to setup the python deps
core.warning('Unable to download and extract the scripts needed for installing the python dependecies');
core.endGroup();
return;
}
// Setup tools
@@ -143,6 +144,7 @@ async function installPythonDeps(codeql) {
// This script tries to install some needed tools in the runner. It should not fail, but if it does
// we just abort the process without failing the action
core.warning('Unable to download and extract the scripts needed for installing the python dependecies');
core.endGroup();
return;
}
// Install dependencies
lib/setup-tracer.js.map
+1 -1
View File
File diff suppressed because one or more lines are too long
src/setup-tracer.ts
+2
View File
@@ -143,6 +143,7 @@ async function installPythonDeps(codeql: CodeQL) {
} catch (e) {
// The download should not fail, but in case it fails we just abort trying to setup the python deps
core.warning('Unable to download and extract the scripts needed for installing the python dependecies');
core.endGroup();
return;
}
@@ -153,6 +154,7 @@ async function installPythonDeps(codeql: CodeQL) {
// This script tries to install some needed tools in the runner. It should not fail, but if it does
// we just abort the process without failing the action
core.warning('Unable to download and extract the scripts needed for installing the python dependecies');
core.endGroup();
return;
}
// Install dependencies