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

Commit 984622c

Browse files
author
azlam-abdulsalam
committed
fix(prepare): add log grouping
1 parent b27ef3b commit 984622c

1 file changed

Lines changed: 17 additions & 8 deletions

File tree

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

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import ExternalDependencyDisplayer from '@dxatscale/sfpowerscripts.core/lib/disp
2525
import ReleaseDefinitionGenerator from '../release/ReleaseDefinitionGenerator';
2626
import ReleaseDefinitionSchema from '../release/ReleaseDefinitionSchema';
2727
import { ZERO_BORDER_TABLE } from '../../ui/TableConstants';
28+
import GroupConsoleLogs from '../../ui/GroupConsoleLogs';
2829

2930
const Table = require('cli-table');
3031

@@ -203,6 +204,9 @@ export default class PrepareImpl {
203204

204205
let artifactFetcher: FetchAnArtifact;
205206
if (this.pool.fetchArtifacts) {
207+
208+
let fetchArtifactsLogGroup = new GroupConsoleLogs(`Fetching Artifacts`);
209+
fetchArtifactsLogGroup.begin();
206210
artifactFetcher = new FetchArtifactSelector(
207211
this.pool.fetchArtifacts.artifactFetchScript,
208212
this.pool.fetchArtifacts.npm?.scope,
@@ -225,17 +229,21 @@ export default class PrepareImpl {
225229
);
226230
}
227231
}
232+
fetchArtifactsLogGroup.end();
228233
} else {
234+
let buildArtifactsLogGroup = new GroupConsoleLogs(`Building Artifacts`);
235+
buildArtifactsLogGroup.begin();
229236
//Build All Artifacts
230-
console.log('\n');
231-
console.log(
237+
SFPLogger.log(`${EOL}`);
238+
SFPLogger.log(
232239
'-------------------------------------WARNING!!!!------------------------------------------------'
233-
);
234-
console.log('Building packages, as script to fetch artifacts was not provided');
235-
console.log('This is not ideal, as the artifacts are built from the current head of the provided branch');
236-
console.log('Pools should be prepared with previously validated packages');
237-
console.log(
238-
'------------------------------------------------------------------------------------------------'
240+
,LoggerLevel.WARN);
241+
SFPLogger.log('Building packages, as script to fetch artifacts was not provided',LoggerLevel.WARN);
242+
SFPLogger.log('This is not ideal, as the artifacts are built from the current head of the provided branch',LoggerLevel.WARN);
243+
SFPLogger.log('Pools should be prepared with previously validated packages',LoggerLevel.WARN);
244+
SFPLogger.log(
245+
'------------------------------------------------------------------------------------------------',
246+
LoggerLevel.WARN
239247
);
240248

241249
let buildProps: BuildProps = {
@@ -261,6 +269,7 @@ export default class PrepareImpl {
261269
await ArtifactGenerator.generateArtifact(generatedPackage, process.cwd(), 'artifacts');
262270
this.artifactFetchedCount++;
263271
}
272+
buildArtifactsLogGroup.end();
264273
}
265274

266275
function isPkgToBeInstalled(pkg, restrictedPackages?: string[]): boolean {

0 commit comments

Comments
 (0)