Skip to content
This repository was archived by the owner on Mar 18, 2024. It is now read-only.

Commit ee3e827

Browse files
azlam-abdulsalamazlam-abdulsalam
andauthored
fix(prepare): ensure prepare in build mode respects releaseconfig (#1281)
Prepare while in build mode was not respecting the passed in release config. This fix is to ensure prepare respects the same Co-authored-by: azlam-abdulsalam <azlam.abdulsalam@accenture.com>
1 parent f08a907 commit ee3e827

File tree

1 file changed

+23
-0
lines changed

1 file changed

+23
-0
lines changed

packages/sfpowerscripts-cli/src/impl/prepare/PrepareImpl.ts

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ import ReleaseDefinitionGenerator from '../release/ReleaseDefinitionGenerator';
2626
import ReleaseDefinitionSchema from '../release/ReleaseDefinitionSchema';
2727
import { ZERO_BORDER_TABLE } from '../../ui/TableConstants';
2828
import GroupConsoleLogs from '../../ui/GroupConsoleLogs';
29+
import ReleaseConfig from '../release/ReleaseConfig';
30+
import { COLOR_KEY_VALUE } from '@dxatscale/sfp-logger';
2931

3032
const Table = require('cli-table');
3133

@@ -259,6 +261,8 @@ export default class PrepareImpl {
259261
currentStage: Stage.PREPARE,
260262
};
261263

264+
buildProps = includeOnlyPackagesAsPerReleaseConfig(this.pool.releaseConfigFile, buildProps);
265+
262266
let buildImpl = new BuildImpl(buildProps);
263267
let { generatedPackages, failedPackages } = await buildImpl.exec();
264268

@@ -284,5 +288,24 @@ export default class PrepareImpl {
284288
if (restrictedPackages) return restrictedPackages.includes(pkg.package);
285289
else return true;
286290
}
291+
292+
293+
function includeOnlyPackagesAsPerReleaseConfig(releaseConfigFilePath:string,buildProps: BuildProps,logger?:Logger): BuildProps {
294+
if (releaseConfigFilePath) {
295+
let releaseConfig:ReleaseConfig = new ReleaseConfig(logger, releaseConfigFilePath);
296+
buildProps.includeOnlyPackages = releaseConfig.getPackagesAsPerReleaseConfig();
297+
printIncludeOnlyPackages(buildProps.includeOnlyPackages);
298+
}
299+
return buildProps;
300+
301+
302+
function printIncludeOnlyPackages(includeOnlyPackages: string[]) {
303+
SFPLogger.log(
304+
COLOR_KEY_MESSAGE(`Build will include the below packages as per inclusive filter`),
305+
LoggerLevel.INFO
306+
);
307+
SFPLogger.log(COLOR_KEY_VALUE(`${includeOnlyPackages.toString()}`), LoggerLevel.INFO);
308+
}
309+
}
287310
}
288311
}

0 commit comments

Comments
 (0)