Skip to content

Commit ef9f76f

Browse files
committed
qa: rename targetReference to baseReference as it is more clear
Signed-off-by: Maximilian Bösing <[email protected]>
1 parent b32eebf commit ef9f76f

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

src/config/app.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ describe('config/app', () => {
5757
ignorePhpPlatformRequirements : {},
5858
additionalComposerArguments : [],
5959
backwardCompatibilityCheck : false,
60-
targetReference : null,
60+
baseReference : null,
6161
});
6262
});
6363

@@ -87,7 +87,7 @@ describe('config/app', () => {
8787
ignorePhpPlatformRequirements : {},
8888
additionalComposerArguments : [],
8989
backwardCompatibilityCheck : true,
90-
targetReference : '1111222233334444aaaabbbbccccdddd',
90+
baseReference : '1111222233334444aaaabbbbccccdddd',
9191
});
9292

9393
process.env = environment;

src/config/app.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ export interface Config {
8686
readonly ignorePhpPlatformRequirements: IgnorePhpPlatformRequirements;
8787
readonly additionalComposerArguments: string[];
8888
readonly backwardCompatibilityCheck: boolean;
89-
readonly targetReference: string|null;
89+
readonly baseReference: string|null;
9090
}
9191
export interface Requirements {
9292
readonly codeChecks: boolean;
@@ -480,7 +480,7 @@ export default function createConfig(
480480
ignorePhpPlatformRequirements : configurationFromFile.ignore_php_platform_requirements ?? {},
481481
additionalComposerArguments : [ ... new Set(configurationFromFile.additional_composer_arguments ?? []) ],
482482
backwardCompatibilityCheck : configurationFromFile.backwardCompatibilityCheck ?? false,
483-
targetReference : process.env.GITHUB_BASE_REF ?? null,
483+
baseReference : process.env.GITHUB_BASE_REF ?? null,
484484
};
485485
}
486486

src/tools.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,15 +51,15 @@ function backwardCompatibilityCheckTool(config: Config): ToolRunningContainerDef
5151
return null;
5252
}
5353

54-
if (config.targetReference === null) {
54+
if (config.baseReference === null) {
5555
return null;
5656
}
5757

5858
return {
5959
// @TODO need to `git fetch baseSha1` from source repo!
6060
executionType : ToolExecutionType.STATIC,
6161
name : 'Backward Compatibility Check',
62-
command : `roave-backward-compatibility-check check --from="${ config.targetReference }" --install-development-dependencies`,
62+
command : `roave-backward-compatibility-check check --from="${ config.baseReference }" --install-development-dependencies`,
6363
filesToCheck : [ 'composer.json' ],
6464
toolType : ToolType.CODE_CHECK,
6565
php : CONTAINER_DEFAULT_PHP_VERSION,

0 commit comments

Comments
 (0)