Skip to content

Commit

Permalink
Lombok: Don't set env var if it's already set
Browse files Browse the repository at this point in the history
If the user has manually configured it, then we let their setting stand.
  • Loading branch information
igfoo committed Aug 18, 2023
1 parent e683046 commit d5b5e18
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions src/init-action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -347,18 +347,19 @@ async function run() {
}

if (config.languages.includes(Language.java)) {
if (await features.getValue(Feature.CodeqlJavaLombokEnabled, codeql)) {
logger.info("Enabling CodeQL Java Lombok support");
core.exportVariable(
"CODEQL_EXTRACTOR_JAVA_RUN_ANNOTATION_PROCESSORS",
"true",
const envVar = "CODEQL_EXTRACTOR_JAVA_RUN_ANNOTATION_PROCESSORS";
if (process.env[envVar]) {
logger.info(
`Environment variable ${envVar} already set. Not en/disabling CodeQL Java Lombok support`,
);
} else if (
await features.getValue(Feature.CodeqlJavaLombokEnabled, codeql)
) {
logger.info("Enabling CodeQL Java Lombok support");
core.exportVariable(envVar, "true");
} else {
logger.info("Disabling CodeQL Java Lombok support");
core.exportVariable(
"CODEQL_EXTRACTOR_JAVA_RUN_ANNOTATION_PROCESSORS",
"false",
);
core.exportVariable(envVar, "false");
}
}

Expand Down

0 comments on commit d5b5e18

Please sign in to comment.