Skip to content

refactor(google-maps): move types reference in maps package - #33773

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:clusterer-refs
Sep 7, 2026
Merged

refactor(google-maps): move types reference in maps package#33773
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:clusterer-refs

Conversation

@crisbeto

@crisbeto crisbeto commented Sep 6, 2026

Copy link
Copy Markdown
Member

We need at least one <reference types="google.maps" preserve="true" /> present in the Google Maps package in order to ensure that the .d.ts we publish has a reference to the types, however internally they need to be stripped.

These changes remove the reference that was randomly places in the clusterer types and will update the internal setup to drop the comment.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Sep 6, 2026
@pullapprove
pullapprove Bot requested review from andrewseguin and ok7sai September 6, 2026 08:57
@pullapprove
pullapprove Bot requested a review from tjshiu September 6, 2026 08:58
@crisbeto
crisbeto force-pushed the clusterer-refs branch 3 times, most recently from a08fdaf to f4baace Compare September 6, 2026 11:40
@crisbeto crisbeto changed the title refactor(google-maps): remove reference from clusterers refactor(google-maps): move types reference in maps package Sep 6, 2026
We need at least one `<reference types="google.maps" preserve="true" />` present in the Google Maps package in order to ensure that the .d.ts we publish has a reference to the types, however internally they need to be stripped.

These changes remove the reference that was randomly places in the clusterer types and will update the internal setup to drop the comment.
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Sep 7, 2026
@crisbeto
crisbeto merged commit dd4acb4 into angular:main Sep 7, 2026
29 of 32 checks passed
@crisbeto

crisbeto commented Sep 7, 2026

Copy link
Copy Markdown
Member Author

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

crisbeto added a commit that referenced this pull request Sep 7, 2026
We need at least one `<reference types="google.maps" preserve="true" />` present in the Google Maps package in order to ensure that the .d.ts we publish has a reference to the types, however internally they need to be stripped.

These changes remove the reference that was randomly places in the clusterer types and will update the internal setup to drop the comment.

(cherry picked from commit dd4acb4)
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: google-maps target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants