Skip to content

Rules angular update#67681

Merged
mattrbeck merged 2 commits intoangular:mainfrom
alan-agius4:rules_angular_update
Mar 13, 2026
Merged

Rules angular update#67681
mattrbeck merged 2 commits intoangular:mainfrom
alan-agius4:rules_angular_update

Conversation

@alan-agius4
Copy link
Contributor

No description provided.

…by Rollup

Explicitly adding an `export {}` to modules containing declare global fixes an issue where Rollup would incorrectly claim that the global variable is not defined in the emitted .d.ts files.

Needed to land the latest rules_angular.
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Mar 13, 2026
@ngbot ngbot bot added this to the Backlog milestone Mar 13, 2026
Update rules_angular to latest sha
@alan-agius4 alan-agius4 force-pushed the rules_angular_update branch from 1692e56 to 468a4b3 Compare March 13, 2026 13:57
@alan-agius4 alan-agius4 added the target: minor This PR is targeted for the next minor release label Mar 13, 2026
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 13, 2026
@alan-agius4 alan-agius4 marked this pull request as ready for review March 13, 2026 14:00
@pullapprove pullapprove bot requested a review from atscott March 13, 2026 14:00
@alan-agius4 alan-agius4 requested review from crisbeto and removed request for josephperrott March 13, 2026 14:00
}

private onMessageError(event: ExtendableMessageEvent): void {
private onMessageError(event: MessageEvent<unknown>): void {
Copy link
Contributor Author

@alan-agius4 alan-agius4 Mar 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The type here was incorrect for TS 6.0 the error is also shown in the IDE but was not surfaced before due to an issue with rules_angular workers which didn't use the workspace TS version.

FYI: @josephperrott

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 13, 2026
@alan-agius4 alan-agius4 removed the request for review from atscott March 13, 2026 15:11
@mattrbeck mattrbeck merged commit 806c9aa into angular:main Mar 13, 2026
26 checks passed
@mattrbeck
Copy link
Member

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4 alan-agius4 deleted the rules_angular_update branch March 13, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants