Skip to content

Commit

Permalink
fix(cli): remove the requirement for app-config when exporting fron…
Browse files Browse the repository at this point in the history
…ted plugins to dynamic. (#1592)

Signed-off-by: David Festal <dfestal@redhat.com>
  • Loading branch information
davidfestal committed Apr 30, 2024
1 parent e860c3b commit 577efff
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 6 deletions.
2 changes: 0 additions & 2 deletions packages/cli/src/lib/bundler/bundlePlugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import fs from 'fs-extra';
import webpack from 'webpack';
import yn from 'yn';

import { resolveBaseUrl } from './config';
import { BundlingPathsOptions, resolveBundlingPaths } from './paths';
import { createScalprumConfig } from './scalprumConfig';
import { DynamicPluginOptions } from './types';
Expand Down Expand Up @@ -38,7 +37,6 @@ export async function buildScalprumBundle(
...options,
checksEnabled: false,
isDev: false,
baseUrl: resolveBaseUrl(options.frontendConfig),
},
);

Expand Down
1 change: 0 additions & 1 deletion packages/cli/src/lib/bundler/scalprumConfig.ts
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,6 @@ export async function createScalprumConfig(
plugins.push(
new webpack.EnvironmentPlugin({
HAS_REACT_DOM_CLIENT: false,
APP_CONFIG: options.frontendAppConfigs,
}),
);

Expand Down
3 changes: 0 additions & 3 deletions packages/cli/src/lib/bundler/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,6 @@ export type BundlingOptions = {
export type DynamicPluginOptions = {
checksEnabled?: boolean;
isDev?: boolean;
frontendConfig: Config;
frontendAppConfigs: AppConfig[];
baseUrl?: URL;
parallelism?: number;
pluginMetadata: PluginBuildMetadata;
};
Expand Down

0 comments on commit 577efff

Please sign in to comment.