Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Commit

Permalink
Merge pull request #6086 from LiskHQ/6083-npm-install
Browse files Browse the repository at this point in the history
Failure in package installation during bootstrapping is not detected - Closes #6083
  • Loading branch information
shuse2 authored Jan 21, 2021
2 parents 1ef0b2f + a3a0d92 commit 235fe9d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion commander/src/bootstrapping/env.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@ import yeoman from 'yeoman-environment';

const env = yeoman.createEnv();

env.register(require.resolve('./generators/init'), 'lisk:init');
env.register(require.resolve('./generators/init_generator'), 'lisk:init');

export { env };
Original file line number Diff line number Diff line change
Expand Up @@ -35,16 +35,18 @@ export default class InitGenerator extends BootstrapGenerator {
}

public install(): void {
this.log('\n');
this.log('Initializing git repository');
this.composeWith(require.resolve('generator-git-init'));

this.log('Installing npm packages');
this.npmInstall();
this.log('\n');
this.log(
'After completion of npm installation run below command to start your blockchain app.\n',
);
this.log(`cd ${this.destinationRoot()}; npm start`);
}

public end(): void {
this.log('\n\n');
this.log('All set! Run below command to start your blockchain app.\n');
this.log(`cd ${this.destinationRoot()}; npm start`);
this.installDependencies({ npm: true, bower: false, yarn: false, skipMessage: false });
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"homepage": "",
"repository": {},
"engines": {
"node": ">=12.20.1",
"npm": ">=6.14.10"
"node": ">=12.13.0 <=12",
"npm": ">=6.12.0"
},
"main": "dist/index.js",
"scripts": {
Expand Down

0 comments on commit 235fe9d

Please sign in to comment.