Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Commit

Permalink
deps(beemo-dev): Update to latest configs.
Browse files Browse the repository at this point in the history
  • Loading branch information
milesj committed Mar 1, 2022
1 parent c9f7052 commit fd4ef40
Show file tree
Hide file tree
Showing 9 changed files with 386 additions and 377 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
"lerna": "^4.0.0",
"packemon": "^1.9.0",
"ts-node": "^10.5.0",
"typescript": "^4.5.4",
"typescript": "^4.6.2",
"webpack": "^5.69.1"
},
"resolutions": {
Expand Down
10 changes: 5 additions & 5 deletions packages/core/src/routines/CreateConfigRoutine.ts
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ export class CreateConfigRoutine<Ctx extends ConfigContext> extends Routine<
break;
}

return Promise.resolve(new Path());
return new Path();
}

/**
Expand Down Expand Up @@ -278,7 +278,7 @@ export class CreateConfigRoutine<Ctx extends ConfigContext> extends Routine<
* Merge multiple configuration sources using the current driver.
*/
@Bind()
async mergeConfigs(context: Ctx, configs: ConfigObject[]): Promise<ConfigObject> {
mergeConfigs(context: Ctx, configs: ConfigObject[]): ConfigObject {
const { driver } = this.options;

this.debug('Merging %s config from %d sources', color.symbol(driver.getName()), configs.length);
Expand All @@ -291,7 +291,7 @@ export class CreateConfigRoutine<Ctx extends ConfigContext> extends Routine<

driver.onMergeConfig.emit([context, config]);

return Promise.resolve(config);
return config;
}

/**
Expand Down Expand Up @@ -331,7 +331,7 @@ export class CreateConfigRoutine<Ctx extends ConfigContext> extends Routine<
this.options.driver.onLoadConsumerConfig.emit([context, config]);
}

return Promise.resolve(configs);
return configs;
}

/**
Expand All @@ -350,7 +350,7 @@ export class CreateConfigRoutine<Ctx extends ConfigContext> extends Routine<
this.options.driver.onLoadProviderConfig.emit([context, sourcePath, config]);
}

return Promise.resolve(configs);
return configs;
}

/**
Expand Down
6 changes: 4 additions & 2 deletions packages/core/src/routines/ScaffoldRoutine.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ export class ScaffoldRoutine extends Routine<unknown, unknown, RoutineOptions> {
createPrompter: this.handlePrompter,
cwd: tool.cwd.path(),
debug: tool.config.debug,
// eslint-disable-next-line @typescript-eslint/no-misused-promises
exec: this.handleExec,
logger: new Logger(console.log),
templates,
Expand All @@ -59,12 +60,13 @@ export class ScaffoldRoutine extends Routine<unknown, unknown, RoutineOptions> {
/**
* Handle shell executions from hygen.
*/
private handleExec = (action: string, input: string) =>
this.executeCommand(action, [], {
private handleExec = async (action: string, input: string) => {
await this.executeCommand(action, [], {
input,
preferLocal: true,
shell: true,
});
};

/**
* Temporary solution until boost supports prompts.
Expand Down
2 changes: 1 addition & 1 deletion packages/core/src/routines/driver/ExecuteCommandRoutine.ts
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ export class ExecuteCommandRoutine extends Routine<unknown, unknown, ExecuteComm
nativeOptions[option] = true;
});

return Promise.resolve(nativeOptions);
return nativeOptions;
}

this.debug('Extracting native options from help output');
Expand Down
2 changes: 1 addition & 1 deletion packages/driver-typescript/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
},
"devDependencies": {
"@boost/test-utils": "^3.0.0",
"typescript": "^4.5.4"
"typescript": "^4.6.2"
},
"peerDependencies": {
"@beemo/core": "^2.0.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -305,6 +305,7 @@ describe('syncProjectRefs()', () => {

it('emits `onCreateProjectConfigFile` event', async () => {
const spy = jest.fn((filePath, config, isTests) => {
// eslint-disable-next-line jest/no-conditional-in-test
if (isTests) {
// eslint-disable-next-line no-param-reassign
config.compilerOptions.testsOnly = true;
Expand Down
2 changes: 1 addition & 1 deletion packages/local/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
"@beemo/driver-webpack": "*"
},
"devDependencies": {
"@beemo/dev": "^1.7.4",
"@beemo/dev": "^1.7.8",
"chalk": "^4.1.2",
"execa": "^5.1.1",
"fast-glob": "^3.2.7",
Expand Down
2 changes: 1 addition & 1 deletion packages/local/src/scripts/RunIntegrationTests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class RunIntegrationTestsScript extends Script<RunIntegrationTestsOptions> {
if (!script) {
console.warn(`Script "integration:${type}" has not been defined for ${name}, skipping.`);

return Promise.resolve();
return undefined;
}

console.log('Testing %s - %s', chalk.yellow(pkg.name), script);
Expand Down
Loading

0 comments on commit fd4ef40

Please sign in to comment.