Skip to content

Commit

Permalink
feat(angular): add backwards compatibility to module-federation-dev-s…
Browse files Browse the repository at this point in the history
…erver (#14180)
  • Loading branch information
Coly010 committed Jan 6, 2023
1 parent 39eba95 commit dd8be60
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ import {
Workspaces,
} from '@nrwl/devkit';
import { scheduleTarget } from 'nx/src/adapter/ngcli-adapter';
import { BuilderContext, createBuilder } from '@angular-devkit/architect';
import { JsonObject } from '@angular-devkit/core';
import { executeWebpackDevServerBuilder } from '../webpack-dev-server/webpack-dev-server.impl';
import { readProjectsConfigurationFromProjectGraph } from 'nx/src/project-graph/project-graph';
import {
Expand All @@ -17,7 +15,7 @@ import {

export function executeModuleFederationDevServerBuilder(
schema: Schema,
context: BuilderContext
context: import('@angular-devkit/architect').BuilderContext
) {
const { ...options } = schema;
const projectGraph = readCachedProjectGraph();
Expand Down Expand Up @@ -95,6 +93,6 @@ export function executeModuleFederationDevServerBuilder(
return executeWebpackDevServerBuilder(options, context);
}

export default createBuilder<JsonObject & Schema>(
export default require('@angular-devkit/architect').createBuilder(
executeModuleFederationDevServerBuilder
);
3 changes: 1 addition & 2 deletions packages/angular/src/builders/utilities/buildable-libs.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import { BuilderContext } from '@angular-devkit/architect';
import {
calculateProjectDependencies,
createTmpTsConfig,
Expand All @@ -9,7 +8,7 @@ import { join } from 'path';

export function createTmpTsConfigForBuildableLibs(
tsConfigPath: string,
context: BuilderContext,
context: import('@angular-devkit/architect').BuilderContext,
target?: string
) {
let dependencies: DependentBuildableProjectNode[];
Expand Down
5 changes: 2 additions & 3 deletions packages/angular/src/builders/utilities/module-federation.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
import { ProjectConfiguration } from 'nx/src/config/workspace-json-project-json';
import { BuilderContext } from '@angular-devkit/architect';
import { join } from 'path';
import { existsSync, readFileSync } from 'fs';
import { Remotes } from '@nrwl/devkit';

export function getDynamicRemotes(
project: ProjectConfiguration,
context: BuilderContext,
context: import('@angular-devkit/architect').BuilderContext,
workspaceProjects: Record<string, ProjectConfiguration>,
remotesToSkip: Set<string>
): string[] {
Expand Down Expand Up @@ -67,7 +66,7 @@ export function getDynamicRemotes(

export function getStaticRemotes(
project: ProjectConfiguration,
context: BuilderContext,
context: import('@angular-devkit/architect').BuilderContext,
workspaceProjects: Record<string, ProjectConfiguration>,
remotesToSkip: Set<string>
): string[] {
Expand Down
3 changes: 1 addition & 2 deletions packages/angular/src/builders/utilities/webpack.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
import type { Target } from '@angular-devkit/architect';
import { merge } from 'webpack-merge';

export async function mergeCustomWebpackConfig(
baseWebpackConfig: any,
pathToWebpackConfig: string,
options: { tsConfig: string; [k: string]: any },
target: Target
target: import('@angular-devkit/architect').Target
) {
const customWebpackConfiguration = resolveCustomWebpackConfig(
pathToWebpackConfig,
Expand Down

0 comments on commit dd8be60

Please sign in to comment.