fixup! perf(@angular/build): overlap library asset resolution with bundling and typechecking

Author: alan-agius4

packages/angular/build/src/builders/library/pipeline/build-action.ts

@@ -129,10 +129,10 @@ export async function buildAction(actionContext: BuildActionContext): Promise<Ex
         return result;
       }
 
+      result.addWarnings(bundleOutput.warnings);
       if (result.hasErrors) {
         return result;
       }
-      result.addWarnings(bundleOutput.warnings);
 
       filesToEmit.push(...bundleOutput.filesToEmit);
       buildState.recordBundleOutput(bundleOutput.bundleResults);

packages/angular/build/src/builders/library/tests/behavior/build_spec.ts

@@ -210,7 +210,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       );
     });
 
-    it('should log failure on build error without secondary bundler warnings', async () => {
+    it('should log failure on build error with bundler warnings', async () => {
       await harness.writeFile(
         'projects/lib/src/public-api.ts',
         `
@@ -239,7 +239,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
           ),
         }),
       );
-      expect(logs).not.toContain(jasmine.objectContaining({ level: 'warn' }));
+      expect(logs).toContain(jasmine.objectContaining({ level: 'warn' }));
       expect(logs).not.toContain(
         jasmine.objectContaining({
           message: jasmine.stringMatching(/Output location:/),