Skip to content

fix(google-maps): implicitly include google.maps types instead of the triple slash workaround #30942

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

josephperrott
Copy link
Member

Previously, due to an issue with the rules_nodejs toolchain we had to include the google.maps namespace types via a triple slash comment with preserve set to true, this changes to instead include the types in the tsconfig as normal

… triple slash workaround

Previously, due to an issue with the rules_nodejs toolchain we had to include the google.maps namespace types via a triple
slash comment with preserve set to true, this changes to instead include the types in the tsconfig as normal
@josephperrott josephperrott added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release area: google-maps area: build & ci Related the build and CI infrastructure of the project labels Apr 23, 2025
@angular-robot angular-robot bot removed the area: build & ci Related the build and CI infrastructure of the project label Apr 23, 2025
@josephperrott josephperrott marked this pull request as ready for review April 23, 2025 19:38
@josephperrott josephperrott requested a review from a team as a code owner April 23, 2025 19:38
@josephperrott josephperrott requested review from mmalerba and andrewseguin and removed request for a team April 23, 2025 19:38
@mmalerba mmalerba removed their request for review April 24, 2025 22:35
@mmalerba mmalerba merged commit cc61697 into angular:main Apr 24, 2025
32 of 34 checks passed
@mmalerba
Copy link
Contributor

The changes were merged into the following branches: main, 19.2.x

mmalerba pushed a commit that referenced this pull request Apr 24, 2025
… triple slash workaround (#30942)

Previously, due to an issue with the rules_nodejs toolchain we had to include the google.maps namespace types via a triple
slash comment with preserve set to true, this changes to instead include the types in the tsconfig as normal

(cherry picked from commit cc61697)
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.

3 participants