Logging: Add a space after a URL for easier copying
This commit is contained in:
parent
222ac621e3
commit
4dca88a5fc
6 changed files with 8 additions and 8 deletions
4
lib/setup-codeql.js
generated
4
lib/setup-codeql.js
generated
|
|
@ -340,10 +340,10 @@ async function getCodeQLSource(toolsInput, defaultCliVersion, apiDetails, varian
|
||||||
(await useZstdBundle(cliVersion, features, tarSupportsZstd)), logger);
|
(await useZstdBundle(cliVersion, features, tarSupportsZstd)), logger);
|
||||||
}
|
}
|
||||||
if (cliVersion) {
|
if (cliVersion) {
|
||||||
logger.info(`Using CodeQL CLI version ${cliVersion} sourced from ${url}.`);
|
logger.info(`Using CodeQL CLI version ${cliVersion} sourced from ${url} .`);
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
logger.info(`Using CodeQL CLI sourced from ${url}.`);
|
logger.info(`Using CodeQL CLI sourced from ${url} .`);
|
||||||
}
|
}
|
||||||
return {
|
return {
|
||||||
bundleVersion: tagName && tryGetBundleVersionFromTagName(tagName, logger),
|
bundleVersion: tagName && tryGetBundleVersionFromTagName(tagName, logger),
|
||||||
|
|
|
||||||
File diff suppressed because one or more lines are too long
2
lib/setup-codeql.test.js
generated
2
lib/setup-codeql.test.js
generated
|
|
@ -161,7 +161,7 @@ ava_1.default.beforeEach(() => {
|
||||||
// bundle contains..
|
// bundle contains..
|
||||||
t.is(result.toolsVersion, expectedVersion);
|
t.is(result.toolsVersion, expectedVersion);
|
||||||
// Ensure message logging CodeQL CLI version was present in user logs.
|
// Ensure message logging CodeQL CLI version was present in user logs.
|
||||||
const expected_message = `Using CodeQL CLI version 2.16.0 sourced from ${bundleUrl}.`;
|
const expected_message = `Using CodeQL CLI version 2.16.0 sourced from ${bundleUrl} .`;
|
||||||
t.assert(loggedMessages.some((msg) => typeof msg.message === "string" &&
|
t.assert(loggedMessages.some((msg) => typeof msg.message === "string" &&
|
||||||
msg.message.includes(expected_message)));
|
msg.message.includes(expected_message)));
|
||||||
});
|
});
|
||||||
|
|
|
||||||
File diff suppressed because one or more lines are too long
|
|
@ -237,7 +237,7 @@ test("setupCodeQLBundle logs the CodeQL CLI version being used when asked to dow
|
||||||
t.is(result.toolsVersion, expectedVersion);
|
t.is(result.toolsVersion, expectedVersion);
|
||||||
|
|
||||||
// Ensure message logging CodeQL CLI version was present in user logs.
|
// Ensure message logging CodeQL CLI version was present in user logs.
|
||||||
const expected_message: string = `Using CodeQL CLI version 2.16.0 sourced from ${bundleUrl}.`;
|
const expected_message: string = `Using CodeQL CLI version 2.16.0 sourced from ${bundleUrl} .`;
|
||||||
t.assert(
|
t.assert(
|
||||||
loggedMessages.some(
|
loggedMessages.some(
|
||||||
(msg) =>
|
(msg) =>
|
||||||
|
|
|
||||||
|
|
@ -446,9 +446,9 @@ export async function getCodeQLSource(
|
||||||
}
|
}
|
||||||
|
|
||||||
if (cliVersion) {
|
if (cliVersion) {
|
||||||
logger.info(`Using CodeQL CLI version ${cliVersion} sourced from ${url}.`);
|
logger.info(`Using CodeQL CLI version ${cliVersion} sourced from ${url} .`);
|
||||||
} else {
|
} else {
|
||||||
logger.info(`Using CodeQL CLI sourced from ${url}.`);
|
logger.info(`Using CodeQL CLI sourced from ${url} .`);
|
||||||
}
|
}
|
||||||
return {
|
return {
|
||||||
bundleVersion: tagName && tryGetBundleVersionFromTagName(tagName, logger),
|
bundleVersion: tagName && tryGetBundleVersionFromTagName(tagName, logger),
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue