robojerk/codeql-action

Merge pull request #2135 from github/henrymercer/no-analyze-config

Browse source 
Avoid analyzing generated configuration file
This commit is contained in:
Henry Mercer 2024-02-13 18:32:43 +00:00 • committed by GitHub
commit bc64d12bb9
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 5 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/config-utils.js generated
View file

@ -462,13 +462,13 @@ function dbLocationOrDefault(dbLocation, tempDir) {
*/
async function initConfig(inputs) {
let config;
const { logger, workspacePath } = inputs;
const { logger, tempDir } = inputs;
// if configInput is set, it takes precedence over configFile
if (inputs.configInput) {
if (inputs.configFile) {
logger.warning(`Both a config file and config input were provided. Ignoring config file.`);
}
inputs.configFile = path.resolve(workspacePath, "user-config-from-action.yml");
inputs.configFile = path.resolve(tempDir, "user-config-from-action.yml");
fs.writeFileSync(inputs.configFile, inputs.configInput);
logger.debug(`Using config from action input: ${inputs.configFile}`);
}

2
lib/config-utils.js.map
View file

File diff suppressed because one or more lines are too long

7
src/config-utils.ts
View file

@ -813,7 +813,7 @@ function dbLocationOrDefault(
export async function initConfig(inputs: InitConfigInputs): Promise<Config> {
let config: Config;
const { logger, workspacePath } = inputs;
const { logger, tempDir } = inputs;
// if configInput is set, it takes precedence over configFile
if (inputs.configInput) {
@ -822,10 +822,7 @@ export async function initConfig(inputs: InitConfigInputs): Promise<Config> {
`Both a config file and config input were provided. Ignoring config file.`,
);
}
inputs.configFile = path.resolve(
workspacePath,
"user-config-from-action.yml",
);
inputs.configFile = path.resolve(tempDir, "user-config-from-action.yml");
fs.writeFileSync(inputs.configFile, inputs.configInput);
logger.debug(`Using config from action input: ${inputs.configFile}`);
}