Commit84f1f63

clydin

authored and

dgp1130

committed

fix(@angular-devkit/build-angular): issue dev-server support warning when using esbuild builder

The dev-server builder currently does not support the experimental esbuild-based browser applicationbuilder and will use the Webpack-based builder instead. To better inform users of this behavior, awarning is now issued upon executing the dev-server.(cherry picked from commit0fcc603)

1 parent1c55d3a commit84f1f63Copy full SHA for 84f1f63

File tree

1 file changed

+10

-0

lines changed

packages/angular_devkit/build_angular/src/builders/dev-server
- index.ts

1 file changed

+10

-0

lines changed

`‎packages/angular_devkit/build_angular/src/builders/dev-server/index.ts‎`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -141,6 +141,16 @@ export function serveWebpackBrowser(`
`141`	`141`	`constcacheOptions=normalizeCacheOptions(metadata,context.workspaceRoot);`
`142`	`142`
`143`	`143`	`constbrowserName=awaitcontext.getBuilderNameForTarget(browserTarget);`
	`144`	`+`
	`145`	`+// Issue a warning that the dev-server does not currently support the experimental esbuild-`
	`146`	`+// based builder and will use Webpack.`
	`147`	`+if(browserName==='@angular-devkit/build-angular:browser-esbuild'){`
	`148`	`+logger.warn(`
	`149`	`+'WARNING: The experimental esbuild-based builder is not currently supported '+`
	`150`	`+'by the dev-server. The stable Webpack-based builder will be used instead.',`
	`151`	`+);`
	`152`	`+}`
	`153`	`+`
`144`	`154`	`constbrowserOptions=(awaitcontext.validateOptions(`
`145`	`155`	`{`
`146`	`156`	`...rawBrowserOptions,`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit84f1f63

File tree

1 file changed

1 file changed

`‎packages/angular_devkit/build_angular/src/builders/dev-server/index.ts‎`

0 commit comments